From 3ce39d2b497ab5356290a22b324181386af51c51 Mon Sep 17 00:00:00 2001
From: Thorbjørn Lindeijer <bjorn@lindeijer.nl>
Date: Tue, 5 Mar 2024 10:13:15 +0100
Subject: 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
---
 src/net/tmwa/playerhandler.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src/net/tmwa/playerhandler.h')

diff --git a/src/net/tmwa/playerhandler.h b/src/net/tmwa/playerhandler.h
index 24284ee4..f2a1b40e 100644
--- a/src/net/tmwa/playerhandler.h
+++ b/src/net/tmwa/playerhandler.h
@@ -29,7 +29,7 @@
 
 namespace TmwAthena {
 
-class PlayerHandler : public MessageHandler, public Net::PlayerHandler
+class PlayerHandler final : public MessageHandler, public Net::PlayerHandler
 {
     public:
         PlayerHandler();
-- 
cgit v1.2.3-70-g09d2