----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41946/#review113033 -----------------------------------------------------------
3rdparty/libprocess/3rdparty/stout/include/stout/representation.hpp (line 66) <https://reviews.apache.org/r/41946/#comment173524> Seems like an `explicit` ctr is good enough here, right? Otherwise `/* implicit */` would show that this is intentional. 3rdparty/libprocess/3rdparty/stout/include/stout/representation.hpp (line 67) <https://reviews.apache.org/r/41946/#comment173526> Any reason you did not delete the ctr from `T&&`? 3rdparty/libprocess/3rdparty/stout/include/stout/representation.hpp (line 76) <https://reviews.apache.org/r/41946/#comment173529> Would it make sense to add an r-value ref qualifier here to effectively make non-temporary `References` unusable? At least it would alleviate some cases of lifetime mismatches between `value` and it's `Representation`. - Benjamin Bannier On Jan. 6, 2016, 3:48 a.m., Michael Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41946/ > ----------------------------------------------------------- > > (Updated Jan. 6, 2016, 3:48 a.m.) > > > Review request for mesos and Benjamin Hindman. > > > Repository: mesos > > > Description > ------- > > See summary. > > > Diffs > ----- > > 3rdparty/libprocess/3rdparty/stout/include/Makefile.am > b2dea9b93adfa3ea0415a0b5c81a369dd29b6cfe > 3rdparty/libprocess/3rdparty/stout/include/stout/representation.hpp > PRE-CREATION > > Diff: https://reviews.apache.org/r/41946/diff/ > > > Testing > ------- > > > Thanks, > > Michael Park > >