> On Jan. 11, 2017, 7 p.m., Michael Park wrote: > > 3rdparty/stout/include/stout/jsonify.hpp, lines 75-78 > > <https://reviews.apache.org/r/55435/diff/2/?file=1603287#file1603287line75> > > > > Shouldn't we call `_configthreadlocale(_DISABLE_PER_THREAD_LOCALE);` > > here to recover the original `setlocale` behavior?
Added comments to explain the proper usage of the locale guard. The destructor now restores the previous config. - Joseph ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55435/#review161347 ----------------------------------------------------------- On Jan. 12, 2017, 3:14 p.m., Joseph Wu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55435/ > ----------------------------------------------------------- > > (Updated Jan. 12, 2017, 3:14 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*`. > > This also adds a comment to the locale guard. > > > 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 > >
