search

yellowstonegames / SquidLib

Useful tools for roguelike, role-playing, strategy, and other grid-based games in Java. Feedback is welcome!
Other
454 stars 46 forks source link

AOE is not Serializable #196

Closed Rakaneth closed 6 years ago

Rakaneth commented 6 years ago

AOE is not Serializable, thus causing Technique to fail to serialize. (see #195)

tommyettinger commented 6 years ago

I was wondering if this chain of things would need to be made Serializable... working on it now.

tommyettinger commented 6 years ago

I'm only closing this tentatively because I don't know if all the AOE implementations fully work with Serializable. DijkstraMap was already Serializable; I think FOV was too (it is now). The latest commit made AOE and all its implementations Serializable, plus LOS, Spill, and SColor for good measure. I'm not sure if any of those classes use objects of some non-Serializable class, though; it's getting less and less likely that some important class is not Serializable though.