search

OutpostUniverse / OP2Utility

C++ library for working with Outpost 2 related files and tasks.
MIT License
4 stars 0 forks source link

Add TrimTilesetSources #222

Closed DanRStevens closed 5 years ago

DanRStevens commented 5 years ago

Add MapData::TrimTilesetSources().

DanRStevens commented 5 years ago

Ahh, good catch on the bug.

You're probably right about removing the tilesetCount field from the map struct, as it has become a derived attribute by virtue of using vectors. Though in the interests of serialization, we probably want to keep the field in the MapHeader struct. The change might be to not use the MapHeader struct inside of the MapData struct. We could also normalize the map width field in the process, so access is similar to height.

This sounds like a separate larger issue. I'll open a new issue to record it.

As always, thank you for the project file updates.

Edit: And yes, I was naturally curious about the space savings :grin: