Re: Review Request 39594: [stout]: Added function to simultaneously query size and mtime of URI.

2015-11-10 Thread Jan Schlicht
> On Nov. 9, 2015, 2:54 p.m., Till Toenshoff wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/net.hpp, line 141 > > > > > > Why not use `nullptr` instead? Is `nullptr` okay to use? Judging from

Re: Review Request 39594: [stout]: Added function to simultaneously query size and mtime of URI.

2015-11-10 Thread Till Toenshoff
> On Nov. 9, 2015, 1:54 p.m., Till Toenshoff wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/net.hpp, line 141 > > > > > > Why not use `nullptr` instead? > > Jan Schlicht wrote: > Is `nullptr` okay

Re: Review Request 39594: [stout]: Added function to simultaneously query size and mtime of URI.

2015-11-10 Thread Benjamin Bannier
> On Nov. 9, 2015, 1:54 p.m., Till Toenshoff wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/net.hpp, line 141 > > > > > > Why not use `nullptr` instead? > > Jan Schlicht wrote: > Is `nullptr` okay

Re: Review Request 39594: [stout]: Added function to simultaneously query size and mtime of URI.

2015-11-10 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39594/ --- (Updated Nov. 10, 2015, 2:07 p.m.) Review request for mesos, Bernd Mathiske,

Re: Review Request 39594: [stout]: Added function to simultaneously query size and mtime of URI.

2015-11-10 Thread Jan Schlicht
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39594/#review105859 --- Ship it!

Re: Review Request 39594: [stout]: Added function to simultaneously query size and mtime of URI.

2015-11-10 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39594/ --- (Updated Nov. 10, 2015, 1:45 p.m.) Review request for mesos, Bernd Mathiske,

Re: Review Request 39594: [stout]: Added function to simultaneously query size and mtime of URI.

2015-11-10 Thread Benjamin Bannier
> On Nov. 9, 2015, 1:54 p.m., Till Toenshoff wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/net.hpp, line 130 > > > > > > Add blank line here please. I would be happy to oblige, but that wouldn't

Re: Review Request 39594: [stout]: Added function to simultaneously query size and mtime of URI.

2015-11-09 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39594/#review105488 --- As discussed, let's re-iterate on the actual function / signature.

Re: Review Request 39594: [stout]: Added function to simultaneously query size and mtime of URI.

2015-11-06 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39594/ --- (Updated Nov. 6, 2015, 11:04 a.m.) Review request for mesos, Bernd Mathiske,

Re: Review Request 39594: [stout]: Added function to simultaneously query size and mtime of URI.

2015-11-06 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39594/ --- (Updated Nov. 6, 2015, 7:52 p.m.) Review request for mesos, Bernd Mathiske,

Review Request 39594: [stout]: Added function to simultaneously query size and mtime of URI.

2015-10-23 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39594/ --- Review request for mesos, Bernd Mathiske, Jan Schlicht, and Till Toenshoff.