Noise: Prevent unittest crash caused by division by zero

[oweals/minetest.git] / src / settings.h

diff --git a/src/settings.h b/src/settings.h
index e570c1a84930d8256924cbc1d482048393587623..f0baf41bfddd5aaa1184b9683e67b2aca929aaa7 100644 (file)
--- a/src/settings.h
+++ b/src/settings.h
@@ -22,10 +22,10 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 
 #include "irrlichttypes_bloated.h"
 #include "util/string.h"
-#include "threading/mutex.h"
 #include <string>
 #include <list>
 #include <set>
+#include <mutex>
 
 class Settings;
 struct NoiseParams;
@@ -73,15 +73,10 @@ struct ValueSpec {
 };
 
 struct SettingsEntry {
-       SettingsEntry() :
-               group(NULL),
-               is_group(false)
-       {}
+       SettingsEntry() {}
 
        SettingsEntry(const std::string &value_) :
-               value(value_),
-               group(NULL),
-               is_group(false)
+               value(value_)
        {}
 
        SettingsEntry(Settings *group_) :
@@ -89,9 +84,9 @@ struct SettingsEntry {
                is_group(true)
        {}
 
-       std::string value;
-       Settings *group;
-       bool is_group;
+       std::string value = "";
+       Settings *group = nullptr;
+       bool is_group = false;
 };
 
 typedef std::unordered_map<std::string, SettingsEntry> SettingEntries;
@@ -139,6 +134,7 @@ public:
        bool getBool(const std::string &name) const;
        u16 getU16(const std::string &name) const;
        s16 getS16(const std::string &name) const;
+       u32 getU32(const std::string &name) const;
        s32 getS32(const std::string &name) const;
        u64 getU64(const std::string &name) const;
        float getFloat(const std::string &name) const;
@@ -232,10 +228,10 @@ private:
 
        SettingsCallbackMap m_callbacks;
 
-       mutable Mutex m_callback_mutex;
+       mutable std::mutex m_callback_mutex;
 
        // All methods that access m_settings/m_defaults directly should lock this.
-       mutable Mutex m_mutex;
+       mutable std::mutex m_mutex;
 
 };