GUI class leaves dangling pointers in fZoneMap when it's moved or copied and destroyed

[oddfacade]

Here. Either these clist* should be turned into smart pointers, the GUI class should implement a copy constructor that deep copies the map, or the default copy constructor should be deleted to force move semantics (and the destructor should be adjusted accordingly). I can make a PR if you'd like. Do you have a preferred solution?