OutpostUniverse / OP2Utility

C++ library for working with Outpost 2 related files and tasks.
MIT License
4 stars 0 forks source link

Consolidate `string` read with generalized container read #296

Open DanRStevens opened 5 years ago

DanRStevens commented 5 years ago

As mentioned in PR #288, the string read method can be consolidated with the generalized container read method. However, this needs C++17 support to enable, as the string.data() method doesn't have a non-const overload until C++17.

Consolidating the two methods will allow the read code to match the consolidation in the corresponding write code.