Re: Review Request 42144: Removed deprecated constructor of http::Unauthorized in libprocess.

2016-01-20 Thread Jan Schlicht
> On Jan. 11, 2016, 4:53 p.m., Jan Schlicht wrote: > > 3rdparty/libprocess/include/process/http.hpp, line 538 > > > > > > Instead of using `std::vector`, we could also use > > `std::initializer_list` here. Same for

Re: Review Request 42144: Removed deprecated constructor of http::Unauthorized in libprocess.

2016-01-11 Thread Alexander Rojas
> On Jan. 11, 2016, 4:24 p.m., Benjamin Bannier wrote: > > 3rdparty/libprocess/include/process/http.hpp, lines 527-546 > > > > > > Sorry for not noticing this earlier, but the constructors are now > > almost

Re: Review Request 42144: Removed deprecated constructor of http::Unauthorized in libprocess.

2016-01-11 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42144/#review113766 --- Ship it! Ship It! - Benjamin Bannier On Jan. 11, 2016, 4:07

Review Request 42144: Removed deprecated constructor of http::Unauthorized in libprocess.

2016-01-11 Thread Alexander Rojas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42144/ --- Review request for mesos, Benjamin Bannier, Ben Mahler, and Till Toenshoff.

Re: Review Request 42144: Removed deprecated constructor of http::Unauthorized in libprocess.

2016-01-11 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42144/#review113763 --- 3rdparty/libprocess/include/process/http.hpp (lines 527 - 546)