wheybags / freeablo

[ARCHIVED] Modern reimplementation of the Diablo 1 game engine
GNU General Public License v3.0
2.16k stars 195 forks source link

Fix/save const correctness #479

Closed grantramsay closed 4 years ago

grantramsay commented 4 years ago

This is a single commit appended on same branch as #475 (because of conflicts), the diff will make more sense after that is merged. Just added const to all save methods I could find. Adding the save method to a common base class (#478) requires that items/actors etc have the same save method signature, and it'd be a bit messy to include this in that PR. Things that requiring saving should probably be implementing an ISaveable interface, but that can be done another day

wheybags commented 4 years ago

Looks good, thanks!