Remove ClmHeader constructors

OutpostUniverse / OP2Utility

C++ library for working with Outpost 2 related files and tasks.

MIT License

4 stars 0 forks source link

Closed DanRStevens closed 5 years ago

DanRStevens commented 5 years ago

Replace non-trivial constructor with static factory method.

@Brett208, please check if this addresses the warnings from Issue #217.

Brett208 commented 5 years ago

Dan, what do you think of this change? If you don't like it, please rollback and merge.

I checked and it does indeed fix the warning. Thanks for pushing this so quickly.

-Brett

DanRStevens commented 5 years ago

Looks good. Nice simplification.