diff options
author | Loïc Blot <nerzhul@users.noreply.github.com> | 2017-06-17 19:11:28 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-06-17 19:11:28 +0200 |
commit | 8f7785771b9e02b1a1daf7a252550d78ea93053d (patch) | |
tree | 7a4e4b524dbc63fed3dac99a3844b634cc621d0d /src/mapsector.cpp | |
parent | 76be103a91d6987527af19e87d93007be8ba8a67 (diff) | |
download | minetest-8f7785771b9e02b1a1daf7a252550d78ea93053d.tar.gz minetest-8f7785771b9e02b1a1daf7a252550d78ea93053d.tar.bz2 minetest-8f7785771b9e02b1a1daf7a252550d78ea93053d.zip |
Cpp11 initializers 2 (#5999)
* C++11 patchset 10: continue cleanup on constructors
* Drop obsolete bool MainMenuData::enable_public (setting is called with cURL in server loop)
* More classes cleanup
* More classes cleanup + change NULL tests to boolean tests
Diffstat (limited to 'src/mapsector.cpp')
-rw-r--r-- | src/mapsector.cpp | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/src/mapsector.cpp b/src/mapsector.cpp index ad7cb7b70..446b43747 100644 --- a/src/mapsector.cpp +++ b/src/mapsector.cpp @@ -23,11 +23,9 @@ with this program; if not, write to the Free Software Foundation, Inc., #include "serialization.h" MapSector::MapSector(Map *parent, v2s16 pos, IGameDef *gamedef): - differs_from_disk(false), m_parent(parent), m_pos(pos), - m_gamedef(gamedef), - m_block_cache(NULL) + m_gamedef(gamedef) { } @@ -39,7 +37,7 @@ MapSector::~MapSector() void MapSector::deleteBlocks() { // Clear cache - m_block_cache = NULL; + m_block_cache = nullptr; // Delete all for (std::unordered_map<s16, MapBlock*>::iterator i = m_blocks.begin(); @@ -55,13 +53,13 @@ MapBlock * MapSector::getBlockBuffered(s16 y) { MapBlock *block; - if (m_block_cache != NULL && y == m_block_cache_y) { + if (m_block_cache && y == m_block_cache_y) { return m_block_cache; } // If block doesn't exist, return NULL std::unordered_map<s16, MapBlock*>::const_iterator n = m_blocks.find(y); - block = (n != m_blocks.end() ? n->second : NULL); + block = (n != m_blocks.end() ? n->second : nullptr); // Cache the last result m_block_cache_y = y; @@ -100,7 +98,7 @@ void MapSector::insertBlock(MapBlock *block) s16 block_y = block->getPos().Y; MapBlock *block2 = getBlockBuffered(block_y); - if(block2 != NULL){ + if (block2) { throw AlreadyExistsException("Block already exists"); } @@ -116,7 +114,7 @@ void MapSector::deleteBlock(MapBlock *block) s16 block_y = block->getPos().Y; // Clear from cache - m_block_cache = NULL; + m_block_cache = nullptr; // Remove from container m_blocks.erase(block_y); |