building_manager.* : Style corrections.

Cpplint suggests to make non-const references in ProblemInstance::ProblemInstance() either const or use pointers instead.

Is it useful/relevant to our implementation?