----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56017/#review167556 -----------------------------------------------------------
Fix it, then Ship it! src/common/protobuf_utils.hpp Lines 96-98 (original), 98-100 (patched) <https://reviews.apache.org/r/56017/#comment239433> The last sentence seems un-necssary because the it's clear from the arguments what fields can be overwritten? - Vinod Kone On Feb. 28, 2017, 3:48 p.m., Alexander Rukletsov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56017/ > ----------------------------------------------------------- > > (Updated Feb. 28, 2017, 3:48 p.m.) > > > Review request for mesos, Gastón Kleiman and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > See summary. > > > Diffs > ----- > > src/common/protobuf_utils.hpp 09e468c77f0cdd931302d1bdcc192370b6ce3340 > src/common/protobuf_utils.cpp 944263bbaa87a65005fd924bccfadb7293312fa0 > > > Diff: https://reviews.apache.org/r/56017/diff/2/ > > > Testing > ------- > > make check > > > Thanks, > > Alexander Rukletsov > >
