Looking good. Only nitpick:
ModelStatus status_;
Why the trailing underscore? No other variable in the class has decoration. If it's this:
void StreamingModel::setStatus(StreamingModel::ModelStatus status)
then change the argument to new_status or something similar.
Regardless of that, this looks fine.
« Back to merge proposal
Looking good. Only nitpick:
ModelStatus status_;
Why the trailing underscore? No other variable in the class has decoration. If it's this:
void StreamingModel: :setStatus( StreamingModel: :ModelStatus status)
then change the argument to new_status or something similar.
Regardless of that, this looks fine.