nit: set off the include of the header associated to the .cpp by one line, so that someone doesn't come along later and 'alphabetize'. Our rule is that the first header should always be the associated .hpp (unless you need basic.h, in which case that comes first).