----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55435/#review161347 -----------------------------------------------------------
3rdparty/stout/include/stout/jsonify.hpp (lines 75 - 78) <https://reviews.apache.org/r/55435/#comment232590> Shouldn't we call `_configthreadlocale(_DISABLE_PER_THREAD_LOCALE);` here to recover the original `setlocale` behavior? - Michael Park On Jan. 11, 2017, 4:56 p.m., Joseph Wu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55435/ > ----------------------------------------------------------- > > (Updated Jan. 11, 2017, 4:56 p.m.) > > > Review request for mesos, Alexander Rojas, Daniel Pravat, Alex Clemmer, and > Michael Park. > > > Repository: mesos > > > Description > ------- > > Changing locales in Windows is mainly accomplished via `setlocale`, > which takes a bit mask (with constants named slightly differently > than in the POSIX headers) and a `char*`. > > > Diffs > ----- > > 3rdparty/stout/include/stout/jsonify.hpp > 3c48046e087de2a66139a31449327fd94c149371 > > Diff: https://reviews.apache.org/r/55435/diff/ > > > Testing > ------- > > Windows: > > msbuild Mesos.sln /p:PreferredToolArchitecture=x64 /m /t:stout-tests > 3rdparty\stout\tests\Debug\stout-tests.exe --gtest_filter="*Json*" > > > Thanks, > > Joseph Wu > >
