diff options
author | Matthias Schiffer <mschiffer@universe-factory.net> | 2015-02-02 01:41:17 +0100 |
---|---|---|
committer | Matthias Schiffer <mschiffer@universe-factory.net> | 2015-02-02 01:41:17 +0100 |
commit | 8c1629af42a10ff7e23bb40100a259bf334a9c60 (patch) | |
tree | 123a427489f9317285996dc24f91c350d793e317 /src/World/Region.cpp | |
parent | 1e5e31581689d372fbf32bfdcc0a89417dbf6b70 (diff) | |
download | MinedMap-8c1629af42a10ff7e23bb40100a259bf334a9c60.tar MinedMap-8c1629af42a10ff7e23bb40100a259bf334a9c60.zip |
Change region reader to a visitor pattern
Not keeping the whole region in memory reduces the needed space
to less than 5MB (from about 140).
Diffstat (limited to 'src/World/Region.cpp')
-rw-r--r-- | src/World/Region.cpp | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/src/World/Region.cpp b/src/World/Region.cpp index fec732c..4551d73 100644 --- a/src/World/Region.cpp +++ b/src/World/Region.cpp @@ -54,7 +54,7 @@ Region::ChunkMap Region::processHeader(const uint8_t header[4096]) { return map; } -Region::Region(const char *filename) { +void Region::visitChunks(const char *filename, ChunkVisitor visitor) { std::ifstream file; file.exceptions(std::ios::failbit | std::ios::badbit); file.open(filename, std::ios::in | std::ios::binary); @@ -68,6 +68,8 @@ Region::Region(const char *filename) { chunkMap = processHeader(header); } + bool seen[SIZE][SIZE] = {}; + size_t i = 1, c = 0; while (!file.eof()) { auto it = chunkMap.find(i); @@ -80,16 +82,19 @@ Region::Region(const char *filename) { size_t x, z, len; std::tie(x, z, len) = it->second; - if (chunks[x][z]) + if (seen[x][z]) throw std::invalid_argument("duplicate chunk"); + seen[x][z] = true; + c++; + uint8_t buffer[len * 4096]; file.read((char *)buffer, len * 4096); - chunks[x][z].reset(new Chunk(Buffer(buffer, len * 4096))); + Chunk chunk(Buffer(buffer, len * 4096)); + visitor(x, z, &chunk); i += len; - c++; } if (c != chunkMap.size()) |