----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68945/#review209305 -----------------------------------------------------------
Ship it! 3rdparty/libprocess/include/process/http.hpp Lines 777 (patched) <https://reviews.apache.org/r/68945/#comment293619> I'd probably consider using `strings::format()` here. - Benno Evers On Oct. 8, 2018, 7:50 a.m., Alexander Rukletsov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68945/ > ----------------------------------------------------------- > > (Updated Oct. 8, 2018, 7:50 a.m.) > > > Review request for mesos and Benno Evers. > > > Repository: mesos > > > Description > ------- > > There is no good reason to provide two c-tors for `MethodNotAllowed`, > with one taking `requestMethod` and one not. Instead, an `Option<>` > can be used. This also removes the need for copy-paste in the c-tor > body. > > > Diffs > ----- > > 3rdparty/libprocess/include/process/http.hpp > cef511a1563a26e170ce7f4a49de12776b4512e7 > > > Diff: https://reviews.apache.org/r/68945/diff/2/ > > > Testing > ------- > > `make check` on various linux distros > > > Thanks, > > Alexander Rukletsov > >