----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55900/#review162982 -----------------------------------------------------------
I remember we have special reason to use `Option<T>` here instead of `const Option<T>&` in previous patches. @alex may remember that, or I missunderstanding something here? - haosdent huang On Jan. 25, 2017, 12:24 a.m., Gastón Kleiman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55900/ > ----------------------------------------------------------- > > (Updated Jan. 25, 2017, 12:24 a.m.) > > > Review request for mesos, Alexander Rukletsov, Anand Mazumdar, haosdent > huang, and Vinod Kone. > > > Bugs: MESOS-6280 > https://issues.apache.org/jira/browse/MESOS-6280 > > > Repository: mesos > > > Description > ------- > > Marked `_taskPid` in the constructor as const, to be consistent with the > other parameters. > > > Diffs > ----- > > src/checks/health_checker.hpp 6e558f2061a9e31157c47d31cb64b3a8568aace3 > src/checks/health_checker.cpp 50aa2858e807b27bbab58a3618f5200cfe4eca9e > > Diff: https://reviews.apache.org/r/55900/diff/ > > > Testing > ------- > > `make check` (macOS and Linux) > > > Thanks, > > Gastón Kleiman > >