Make AreaStore cache setting private
authorShadowNinja <shadowninja@minetest.net>
Fri, 30 Oct 2015 03:17:44 +0000 (23:17 -0400)
committerShadowNinja <shadowninja@minetest.net>
Mon, 7 Mar 2016 21:33:20 +0000 (16:33 -0500)
src/areastore.cpp
src/areastore.h

index c2cc4ce97af4ad59ff0835d72a04e9211d3c2f2c..b1b5c0db46088066eb0b74aa66fab3e32dafc71c 100644 (file)
@@ -112,14 +112,14 @@ void AreaStore::serialize(std::ostream &os) const
 
 void AreaStore::invalidateCache()
 {
-       if (cache_enabled) {
+       if (m_cache_enabled) {
                m_res_cache.invalidate();
        }
 }
 
 void AreaStore::setCacheParams(bool enabled, u8 block_radius, size_t limit)
 {
-       cache_enabled = enabled;
+       m_cache_enabled = enabled;
        m_cacheblock_radius = MYMAX(block_radius, 16);
        m_res_cache.setLimit(MYMAX(limit, 20));
        invalidateCache();
@@ -148,7 +148,7 @@ void AreaStore::cacheMiss(void *data, const v3s16 &mpos, std::vector<Area *> *de
 
 void AreaStore::getAreasForPos(std::vector<Area *> *result, v3s16 pos)
 {
-       if (cache_enabled) {
+       if (m_cache_enabled) {
                v3s16 mblock = getContainerPos(pos, m_cacheblock_radius);
                const std::vector<Area *> *pre_list = m_res_cache.lookupCache(mblock);
 
index de458870642fa38cb868d0084c1c25b2a3ac8357..493b4fc25756fa075d33ea6feac063d2373ef45d 100644 (file)
@@ -62,7 +62,6 @@ protected:
 
        // TODO change to unordered_map when we can
        std::map<u32, Area> areas_map;
-       bool cache_enabled; // don't write to this from subclasses, only read.
 public:
        // Updates the area's ID
        virtual bool insertArea(Area *a) = 0;
@@ -83,10 +82,10 @@ public:
        {}
 
        AreaStore() :
-               cache_enabled(true),
                m_cacheblock_radius(64),
                m_res_cache(1000, &cacheMiss, this),
-               m_next_id(0)
+               m_next_id(0),
+               m_cache_enabled(true)
        {
        }
 
@@ -103,7 +102,7 @@ private:
        u8 m_cacheblock_radius; // if you modify this, call invalidateCache()
        LRUCache<v3s16, std::vector<Area *> > m_res_cache;
        u32 m_next_id;
-
+       bool m_cache_enabled;
 };