Elsewhere, we've just defined these in the module that defines the class.  Is there a good reason for defining them here?  (I know we do this in C code, but there we combine the declaration with the definition - here we can't due to C++ restrictions.)