Removed some 'const' keywords from value returns

It makes no sense to mark a return value as const when it is returned by
value.  This only makes sense if the return value is passed by reference, in
order to prevent the receiver from modifying the value.

Also made some other small adjustments. A std::string does not need to be
initialized to "" explicitly and getSafeUtf8String could take its parameter by
reference.

Reviewed-by: Yohann Ferreira

1 files changed, 3 insertions, 3 deletions

diff --git a/src/map.h b/src/map.h
index 8bd47b45..b7917275 100644
--- a/src/map.h
+++ b/src/map.h
@@ -275,13 +275,13 @@ class Map : public Properties
          */
         Vector getTileCenter(int x, int y);
 
-        const std::string getMusicFile() const;
-        const std::string getName() const;
+        std::string getMusicFile() const;
+        std::string getName() const;
 
         /**
          * Gives the map id based on filepath (ex: 009-1)
          */
-        const std::string getFilename() const;
+        std::string getFilename() const;
 
         /**
          * Check the current position against surrounding blocking tiles, and