----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46687/#review139483 -----------------------------------------------------------
Fix it, then Ship it! Could you specify that this only updates `http_tests.cpp`? There are other places in the codebase where we explicitly construct a `std::function` first, but we don't update all of those in this patch. 3rdparty/libprocess/src/tests/http_tests.cpp (lines 192 - 194) <https://reviews.apache.org/r/46687/#comment204738> This fits in one line. 3rdparty/libprocess/src/tests/http_tests.cpp (lines 264 - 266) <https://reviews.apache.org/r/46687/#comment204739> Also fits in one line. - Michael Park On June 24, 2016, 8:57 a.m., haosdent huang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46687/ > ----------------------------------------------------------- > > (Updated June 24, 2016, 8:57 a.m.) > > > Review request for mesos, Kevin Klues and Michael Park. > > > Bugs: MESOS-4611 > https://issues.apache.org/jira/browse/MESOS-4611 > > > Repository: mesos > > > Description > ------- > > Updated legacy `dispatch` calls to use lambda. > > > Diffs > ----- > > 3rdparty/libprocess/src/tests/http_tests.cpp > 4337e6028a3a6e5279793c7c6f73bb9a4f60cb0a > > Diff: https://reviews.apache.org/r/46687/diff/ > > > Testing > ------- > > > Thanks, > > haosdent huang > >
