General code cleanups

* Don't needlessly store or return raw pointers in BeingInfo
* Less copying, more moving
* Less else after return
* Make AddDEF a template instead of a macro
* Removed some unused includes
* Use range-based for loops

1 files changed, 3 insertions, 3 deletions

diff --git a/src/playerrelations.h b/src/playerrelations.h
index 9c696105..a836a976 100644
--- a/src/playerrelations.h
+++ b/src/playerrelations.h
@@ -71,7 +71,7 @@ public:
     std::string mDescription;
     std::string mShortName;
 
-    virtual ~PlayerIgnoreStrategy() {}
+    virtual ~PlayerIgnoreStrategy() = default;
 
     /**
      * Handle the ignoring of the indicated action by the indicated player.
@@ -82,8 +82,8 @@ public:
 class PlayerRelationsListener
 {
 public:
-    PlayerRelationsListener() { }
-    virtual ~PlayerRelationsListener() { }
+    PlayerRelationsListener() = default;
+    virtual ~PlayerRelationsListener() = default;
 
     virtual void updatedPlayer(const std::string &name) = 0;
 };