----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43240/#review118035 -----------------------------------------------------------
Fix it, then Ship it! src/log/tool/benchmark.cpp (line 209) <https://reviews.apache.org/r/43240/#comment179313> can you use 0xff here? - Jie Yu On Feb. 5, 2016, 3:28 a.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43240/ > ----------------------------------------------------------- > > (Updated Feb. 5, 2016, 3:28 a.m.) > > > Review request for mesos and Jie Yu. > > > Repository: mesos > > > Description > ------- > > Whether `char` is signed or not is up to the implementation, so be more > explicit > here. Also, since we want a `char` with all bits set, use a value > representation > that more clearly communicates that. > > > Diffs > ----- > > src/log/tool/benchmark.cpp 9cb60bd66f53e0d191d96a429bfca89d91412339 > > Diff: https://reviews.apache.org/r/43240/diff/ > > > Testing > ------- > > When compiling with trunk clang, compilation fails with > > ../../src/log/tool/benchmark.cpp|209 col 47 warning| implicit conversion > from 'int' to 'value_type' (aka 'char') changes value from 255 to -1 > [-Wconstant-conversion] > > After inserting the explicit cast the code compiles fine. > > > Thanks, > > Benjamin Bannier > >
