General code cleanups

* Use default member initializers
* Use range-based for loops
* Avoid needless pointer references for ShopItem::mDuplicates
* Removed type aliases that are only used once or twice
* Removed more unused includes
* Removed some unused functions
* Removed superfluous .c_str()
* Rely on default copy and assignment operators for Vector class
* Use std::unique_ptr in some places
* Removed duplicated mPlayerMoney updating in SellDialog
* Removed duplicated Game::handleInput call
* Removed unused SDLInput::mMouseInWindow
* Removed remnant of manual widget positioning in HelpWindow
* Removed superfluous initialization of static pointers

1 files changed, 1 insertions, 7 deletions

diff --git a/src/net/tmwa/gui/partytab.cpp b/src/net/tmwa/gui/partytab.cpp
index 89f3a769..6f1e3b68 100644
--- a/src/net/tmwa/gui/partytab.cpp
+++ b/src/net/tmwa/gui/partytab.cpp
@@ -42,10 +42,6 @@ PartyTab::PartyTab() :
     setTabColor(&Theme::getThemeColor(Theme::PARTY_CHAT_TAB));
 }
 
-PartyTab::~PartyTab()
-{
-}
-
 void PartyTab::handleInput(const std::string &msg)
 {
     Net::getPartyHandler()->chat(msg);
@@ -197,9 +193,7 @@ bool PartyTab::handleCommand(const std::string &type, const std::string &args)
 
 void PartyTab::getAutoCompleteList(std::vector<std::string> &names) const
 {
-    Party *p = local_player->getParty();
-
-    if (p)
+    if (Party *p = local_player->getParty())
         p->getNames(names);
 }