summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorBřetislav Štec <valsiterb@gmail.com>2015-08-02 15:08:39 +0200
committerest31 <MTest31@outlook.com>2015-08-02 19:42:22 +0200
commitabe6c072d66777b171399f42548bad75f618f1a3 (patch)
treeeca75356bf3988b2c57e92761db6859aa9e83f27 /src
parent060e56b24c21c4cda76fcac15eecbdce9682d16a (diff)
downloadminetest-abe6c072d66777b171399f42548bad75f618f1a3.tar.gz
minetest-abe6c072d66777b171399f42548bad75f618f1a3.tar.bz2
minetest-abe6c072d66777b171399f42548bad75f618f1a3.zip
src/util/numeric.{cpp,h}: Fix FacePositionCache data race
Diffstat (limited to 'src')
-rw-r--r--src/util/numeric.cpp4
-rw-r--r--src/util/numeric.h2
2 files changed, 6 insertions, 0 deletions
diff --git a/src/util/numeric.cpp b/src/util/numeric.cpp
index 4ddfede92..3fd1c9cf9 100644
--- a/src/util/numeric.cpp
+++ b/src/util/numeric.cpp
@@ -23,13 +23,17 @@ with this program; if not, write to the Free Software Foundation, Inc.,
#include "log.h"
#include "../constants.h" // BS, MAP_BLOCKSIZE
#include "../noise.h" // PseudoRandom, PcgRandom
+#include "../jthread/jmutexautolock.h"
#include <string.h>
#include <iostream>
std::map<u16, std::vector<v3s16> > FacePositionCache::m_cache;
+JMutex FacePositionCache::m_cache_mutex;
// Calculate the borders of a "d-radius" cube
+// TODO: Make it work without mutex and data races, probably thread-local
std::vector<v3s16> FacePositionCache::getFacePositions(u16 d)
{
+ JMutexAutoLock cachelock(m_cache_mutex);
if (m_cache.find(d) != m_cache.end())
return m_cache[d];
diff --git a/src/util/numeric.h b/src/util/numeric.h
index 0eba01359..9fe08434f 100644
--- a/src/util/numeric.h
+++ b/src/util/numeric.h
@@ -24,6 +24,7 @@ with this program; if not, write to the Free Software Foundation, Inc.,
#include "../irr_v2d.h"
#include "../irr_v3d.h"
#include "../irr_aabb3d.h"
+#include "../jthread/jmutex.h"
#include <list>
#include <map>
#include <vector>
@@ -41,6 +42,7 @@ public:
private:
static void generateFacePosition(u16 d);
static std::map<u16, std::vector<v3s16> > m_cache;
+ static JMutex m_cache_mutex;
};
class IndentationRaiser