Terasology / DynamicCities

Cities that get fancy!
18 stars 14 forks source link

chore(BlockBufferSystem): reduce log spam #101

Closed skaldarnar closed 3 years ago

skaldarnar commented 3 years ago

Problem: The state of the block buffer was logged at INFO level with each iteration.

Solution: Reduce the log level to DEBUG and only log if the level is enabled. As removed and the size of blocksToPlace were always the same additional field removed is removed.

While at it, I tried to improve the readability of BlockBufferSystem#setBlocks by making use of Java 8 Stream API. Additionally, add a couple of docstrings, TODOs or FIXMEs here and there to prepare for a future module overhaul.