Qwarkster

Something like "RenderableLayer" would represent layers that would draw down when calling RenderVisibleLayers(). This goes alongside some comments I had seen on another TMX parser where it was brought up...

Conceptually on loading, we have to loop through the first level "children" under the map element, numbering them off as we come across them, regardless of what type of element....

Have to ensure that it doesn't group all the layers, all the objectgroups, all the imagelayers, etc so they can intermingle. I'm sure it can be done with the base...

A simple thought on this, there's no reason to omit values from the save file if it's the same as the default value. Writing out all the field values when...

I noticed this is what is affecting objects not having some values set, due to a lack of Unmarshal method. Anyways I can't offer up anything concrete yet, only been...

I would be wary of allowing fields to be undefined, as that feels like an unnecessary complication, in particular for bool fields. It's good practice to accommodate a vast majority...

A flag, and appropriate checks where it's needed is likely the proper solution. By itself, it's not a particular issue, more of an annoyance to get used to if you...