----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59110/#review174382 -----------------------------------------------------------
Ship it! Ship It! - Neil Conway On May 9, 2017, 8:32 p.m., Andrew Schwartzmeyer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59110/ > ----------------------------------------------------------- > > (Updated May 9, 2017, 8:32 p.m.) > > > Review request for mesos, Li Li and Neil Conway. > > > Bugs: MESOS-7484 > https://issues.apache.org/jira/browse/MESOS-7484 > > > Repository: mesos > > > Description > ------- > > The test case `u8"1.0.0-b\u00e9ta"` in `VersionTest.ParseInvalid` caused > an abort in `isctype.cpp` with VS 2017. Changing the `char` to an > `unsigned char` prevents this. > > > Diffs > ----- > > 3rdparty/stout/include/stout/version.hpp > 9120e42eedcfb4fc5ee41f08354441bc10914baf > > > Diff: https://reviews.apache.org/r/59110/diff/1/ > > > Testing > ------- > > Build and ran `stout-tests` target on Windows. `make check` in > `build/3rdparty/stout` on Linux. > > ``` > [ RUN ] VersionTest.ParseValid > [ OK ] VersionTest.ParseValid (0 ms) > ``` > > > Thanks, > > Andrew Schwartzmeyer > >