----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55296/#review161047 -----------------------------------------------------------
Ship it! Ship It! - Alexander Rojas On Jan. 10, 2017, 1:36 a.m., Michael Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55296/ > ----------------------------------------------------------- > > (Updated Jan. 10, 2017, 1:36 a.m.) > > > Review request for mesos, Alexander Rojas and Joris Van Remoortere. > > > Repository: mesos > > > Description > ------- > > The printing logic for `json.hpp` and `jsonify.hpp` are currently > duplicated. We can reduce this duplication by leveraging `jsonify` for > the implementation of `operator<<` for `JSON::*`. Since `JSON::Value`s > are not generally used for printing, there should be no performance > concerns here. > > > Diffs > ----- > > 3rdparty/stout/include/stout/json.hpp > 62ce15274677112d142a3c829b4a9f06258c9e2c > > Diff: https://reviews.apache.org/r/55296/diff/ > > > Testing > ------- > > > Thanks, > > Michael Park > >
