> On Jan. 6, 2016, 9:51 a.m., Benjamin Bannier wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/representation.hpp, line 66 > > <https://reviews.apache.org/r/41946/diff/2/?file=1183573#file1183573line66> > > > > Seems like an `explicit` ctr is good enough here, right? Otherwise `/* > > implicit */` would show that this is intentional.
I used `reference_wrapper` to implement this, which has an implicit constructor. I think its default behavior is fine for us? - Michael ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41946/#review113033 ----------------------------------------------------------- On Jan. 6, 2016, 10 p.m., Michael Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41946/ > ----------------------------------------------------------- > > (Updated Jan. 6, 2016, 10 p.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 > >
