----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46504/#review130103 ----------------------------------------------------------- For posterity, could you please explain a bit in the description what you intention here is and why you take this implemntation approach? 3rdparty/libprocess/include/process/http.hpp (lines 569 - 571) We usually insert two blank lines here. 3rdparty/libprocess/include/process/http.hpp (line 571) Does it make sense to declare this struct inside `MethodNotAllowed`? 3rdparty/libprocess/include/process/http.hpp (lines 573 - 574) It looks like you don't need to keep this two valus as fields, right? 3rdparty/libprocess/include/process/http.hpp (line 579) Why do you take `Request` instead of a method? 3rdparty/libprocess/include/process/http.hpp (lines 603 - 605) We usually insert two blank lines here. 3rdparty/libprocess/include/process/http.hpp (lines 623 - 624) Blank line, please! 3rdparty/libprocess/include/process/http.hpp Why have you removed this line? - Alexander Rukletsov On April 22, 2016, 3:11 p.m., Jacob Janco wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46504/ > ----------------------------------------------------------- > > (Updated April 22, 2016, 3:11 p.m.) > > > Review request for mesos and Alexander Rukletsov. > > > Bugs: MESOS-4126 > https://issues.apache.org/jira/browse/MESOS-4126 > > > Repository: mesos > > > Description > ------- > > See summary. > > > Diffs > ----- > > 3rdparty/libprocess/include/process/http.hpp 8f4eabcbb71ead1f5c28e1d8a2dd40db7af1f297 > > Diff: https://reviews.apache.org/r/46504/diff/ > > > Testing > ------- > > > Thanks, > > Jacob Janco > >