----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58095/#review174453 -----------------------------------------------------------
src/master/http.cpp Lines 3489-3497 (original), 3491-3499 (patched) <https://reviews.apache.org/r/58095/#comment247624> you can do something like: ```c++ Master *master = this->master; return _roles(principal) .then(defer(master->self(), [master, request](...) { ... ``` and that avoids having to capture `this`. - Alexander Rojas On May 8, 2017, 9:41 a.m., Jay Guo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58095/ > ----------------------------------------------------------- > > (Updated May 8, 2017, 9:41 a.m.) > > > Review request for mesos, Adam B, Alexander Rojas, Benjamin Mahler, and > Michael Park. > > > Bugs: MESOS-4732 and MESOS-7260 > https://issues.apache.org/jira/browse/MESOS-4732 > https://issues.apache.org/jira/browse/MESOS-7260 > > > Repository: mesos > > > Description > ------- > > Refactored `Master::Http::roles` to use `jsonify`. > > > Diffs > ----- > > src/common/http.hpp 93d6088e97c2384f9f6d26e010a501abf2deb43e > src/common/http.cpp 167dce2b9a2d3b68a1df5b4079f701482d34db28 > src/master/http.cpp e2590a17044ac019b24a24629428d4ec8adc0c31 > > > Diff: https://reviews.apache.org/r/58095/diff/5/ > > > Testing > ------- > > no functional changes > make check > > > Thanks, > > Jay Guo > >
