----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40978/#review109492 -----------------------------------------------------------
src/tests/fetcher_tests.cpp (line 278) <https://reviews.apache.org/r/40978/#comment169064> You are right, but all other process constructors use the syntax used here. I suggest we leave it for now and discuss changing this offline (from this ticket). src/tests/fetcher_tests.cpp (line 295) <https://reviews.apache.org/r/40978/#comment169063> Unfortunately there is no precedence for this. All other code like this uses get(). I think this is good enough because it is clear that this is a fresh instance that gets used here. - Bernd Mathiske On Dec. 4, 2015, 10:49 a.m., Joseph Wu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40978/ > ----------------------------------------------------------- > > (Updated Dec. 4, 2015, 10:49 a.m.) > > > Review request for mesos, Bernd Mathiske, Ben Mahler, and Artem Harutyunyan. > > > Repository: mesos > > > Description > ------- > > This addresses the comment here: > https://reviews.apache.org/r/40501/#comment168391 > > > Diffs > ----- > > src/tests/fetcher_tests.cpp 069c9bc6bc54d5bc1275c55e329457651d3c7b71 > > Diff: https://reviews.apache.org/r/40978/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Joseph Wu > >