Re: Review Request 39007: libprocess: Used thread-safe replacement for strerror.

2015-11-13 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39007/#review106444 --- Ship it! Looks good, we won't need the os/strerror.hpp includes

Re: Review Request 39007: libprocess: Used thread-safe replacement for strerror.

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

Re: Review Request 39007: libprocess: Used thread-safe replacement for strerror.

2015-11-12 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39007/#review106256 --- Patch looks great! Reviews applied: [39005, 39007] All tests

Re: Review Request 39007: libprocess: Used thread-safe replacement for strerror.

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

Re: Review Request 39007: libprocess: Used thread-safe replacement for strerror.

2015-11-03 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39007/#review105017 --- Patch looks great! Reviews applied: [39005, 39006, 39007] All

Re: Review Request 39007: libprocess: Used thread-safe replacement for strerror.

2015-10-22 Thread Benjamin Bannier
> On Oct. 21, 2015, 10:24 p.m., Kapil Arya wrote: > > 3rdparty/libprocess/src/io.cpp, line 27 > > > > > > This should at the very end of stout includes. Let's keep lexicographical sort with an ASCII code page (which

Re: Review Request 39007: libprocess: Used thread-safe replacement for strerror.

2015-10-22 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39007/ --- (Updated Oct. 22, 2015, 7:50 a.m.) Review request for mesos, Bernd Mathiske,

Re: Review Request 39007: libprocess: Used thread-safe replacement for strerror.

2015-10-21 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39007/#review103486 --- 3rdparty/libprocess/src/io.cpp (line 27)

Re: Review Request 39007: libprocess: Used thread-safe replacement for strerror.

2015-10-21 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39007/#review103463 --- Ship it! 3rdparty/libprocess/src/profiler.cpp (line 88)

Re: Review Request 39007: libprocess: Used thread-safe replacement for strerror.

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

Re: Review Request 39007: libprocess: Used thread-safe replacement for strerror.

2015-10-08 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39007/#review101988 --- Patch looks great! Reviews applied: [39005, 39007] All tests

Re: Review Request 39007: libprocess: Used thread-safe replacement for strerror.

2015-10-08 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39007/ --- (Updated Oct. 8, 2015, 6:15 p.m.) Review request for mesos, Bernd Mathiske,