General code cleanups

* Use final for all message handlers, Client, LocalPlayer,
  Being::getType, Being::setPosition and Being::setMap.
  (avoids some warnings about virtual dispatch in constructors)
* Use auto in more places
* Use emplace_back instead of push_back in some places
* Use default member initializers
* Less else after return
* Removed superfluous .c_str()
* Removed type aliases that are only used once
* Removed more unused includes

1 files changed, 1 insertions, 5 deletions

diff --git a/src/particle.cpp b/src/particle.cpp
index 3e958417..3062de51 100644
--- a/src/particle.cpp
+++ b/src/particle.cpp
@@ -237,12 +237,8 @@ bool Particle::update()
             p = mChildParticles.erase(p);
         }
     }
-    if (mAlive != ALIVE && mChildParticles.empty() && mAutoDelete)
-    {
-        return false;
-    }
 
-    return true;
+    return !(mAlive != ALIVE && mChildParticles.empty() && mAutoDelete);
 }
 
 void Particle::moveBy(const Vector &change)