Agreed, but I would take this further. To call this C++ and be using character arrays for strings rather than std::string and performing manual memory managemen

Agreed, but I would take this further. To call this C++ and be using character arrays for strings rather than std::string and performing manual memory management suggests this is "C with Classes" at best, notwithstanding the fact that it HEAP allocates a bunch of stuff that it never reclaims. I understand this isn't a code review and I'm not trying to be critical but it's saddens me to see poor C++ code being shown online because that's partly responsible for the bad rap of the language - I know there are other reasons, but this doesn't help.