----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51877/#review148940 -----------------------------------------------------------
I didn't add additional comments because the conversion is intentionally implicit. I guess we can clarify further by saying "Note that `Resource` objects are implicitly converted to `Resource_`" here but this is true elsewhere in resources.cpp as well. - Jiang Yan Xu On Sept. 13, 2016, 7:03 p.m., Guangya Liu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51877/ > ----------------------------------------------------------- > > (Updated Sept. 13, 2016, 7:03 p.m.) > > > Review request for mesos, Benjamin Mahler and Jiang Yan Xu. > > > Repository: mesos > > > Description > ------- > > The API for += and -= `Resource` object are now removed, we should > update the comments for += and -= `Resource_` object by telling > end user for how to convert a `Resource` object to a `Resource_` > object when += and -= a `Resource` object. > > > Diffs > ----- > > include/mesos/resources.hpp 7ba422d57d47777e058c682df9aa780557782423 > include/mesos/v1/resources.hpp add48c7baf8fabf5cc443d60a6c96d2902fc67de > > Diff: https://reviews.apache.org/r/51877/diff/ > > > Testing > ------- > > make > make check > > > Thanks, > > Guangya Liu > >
