worldforge/metaserver-ng — comment on commit ad71d6f0153e48661677866a11689dad89e675ce
File:  line 0
By alriddoch:

Before I make more comments I should say you are definitely moving in the right direction, and the work you did yesterday was good progress. If my comments sound critical I apologise. I am feeling energised and enthused by the direction this code is moving, and the new feature work it will enable long term, so I am keen to help the code improve.

Symbols in headers, including headers guards should not begin with underscores, as those names are reserved for the system and the C library. A better name for you header guard would be METASERVER_HPP or METASERVERNG_METASERVER_HPP depending on how cautious you want to be about keeping names specific.