----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19988/#review39437 -----------------------------------------------------------
3rdparty/libprocess/src/decoder.hpp <https://reviews.apache.org/r/19988/#comment71837> we wrap comments at 70 chars. - Vinod Kone On April 3, 2014, 6:26 a.m., Adam B wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/19988/ > ----------------------------------------------------------- > > (Updated April 3, 2014, 6:26 a.m.) > > > Review request for mesos, Benjamin Hindman and Ben Mahler. > > > Bugs: MESOS-1181 > https://issues.apache.org/jira/browse/MESOS-1181 > > > Repository: mesos-git > > > Description > ------- > > Enabled readability/todo rule for cpplint, which covers the following: > 'Missing username in TODO; it should look like "// TODO(my_username): Stuff."' > > Used git blame to trace back the unaccredited TODOs, 3 to benh and 2 to > bmahler. > bmahler's TODO in slave.cpp is already covered by one in messages.proto for > ReregisterSlaveMessage, so I removed it. > > > Diffs > ----- > > 3rdparty/libprocess/src/decoder.hpp 88da57f > src/exec/exec.cpp 346e39f > src/master/master.cpp 3c3c989 > src/sched/sched.cpp 3684cfe > src/slave/slave.cpp a356f5f > support/mesos-style.py edd41f926d6df30513f8fa6c928960e6495af70a > > Diff: https://reviews.apache.org/r/19988/diff/ > > > Testing > ------- > > Ran mesos-style.py over entire source base with readability/todo enabled. > > > Thanks, > > Adam B > >
