Fix unnecessary exception use in 3 more methods (#8791)
authorJozef Behran <jozuejozef@gmail.com>
Tue, 13 Aug 2019 18:02:50 +0000 (20:02 +0200)
committerSmallJoker <SmallJoker@users.noreply.github.com>
Tue, 13 Aug 2019 18:02:50 +0000 (20:02 +0200)
commitbf22184d6e16f21b9001322aad1bf8f6dbaa372b
treea1650bf7d3fc0cd756a9bbda07b28480b3f6c439
parent72b7a957af331c7481a60705d1a35a78b5afb494
Fix unnecessary exception use in 3 more methods (#8791)

* Fix unnecessary exception use in Server::SendBlocks

The code in this method calls getBlockNoCreate and then
messes around with try...catch to skip blocks which are not
in the memory. Additionally, it repeatedly calls
m_env.getMap() inside this loop. Speed the code up by
extracting the m_env.getMap() out of the loop and getting
rid of the try...catch.

* Fix unnecessary exception use in Server::SendBlock

Another unnecessary try...catch is slowing down
Server::SendBlock. Remove that to speed it up and get a nice
side effect of simplifying the code in question.

* Fix unnecessary exception use in MMVManip::initialEmerge

Remove another unneeded exception usage from
MMVManip::initialEmerge to speed that code up and simplify
it but be careful to not remove the braces as there is a
TimeTaker in use there.
src/map.cpp
src/server.cpp