[Beaness]

Refactor of 1.8 chunk handling, reduces allocations by quite a bit. Deprecates NetworkChunkData chunksToData(Chunk_v1_8[] chunks, byte[] biomes) as it's entirely skipped now to reduce memory copies.

[booky10]

Looks good, and also added some minor changes/improvements

---

0002f2c#diff-f324d7963c6177fb5a6763389a67c36b842d3a40a09530c32a5f916dab21c1a7R155
Don't know much about 1.8 chunk format, but this seemed like a bug to me - can you confirm this?

[Beaness]

The previous !fullChunk || !chunk.isEmpty() matches vanilla: groundUp packets can skip empty sections because the client clears every section missing from the mask. Non-groundUp packets only replace the sections in the mask, so empty ones have to be included.

Also, the light loops in writeChunkData don't check chunkMask anymore, so sections left out of the mask still get their light written.