Re: Review Request 39330: libprocess: Replaced usage of "volatile" with std::atomic.

2015-10-18 Thread Neil Conway
> On Oct. 17, 2015, 8:43 p.m., Joris Van Remoortere wrote: > > 3rdparty/libprocess/include/process/owned.hpp, line 82 > > > > > > Wish we could use `nullptr`, but we currently only allow `NULL` for now. > > Here

Re: Review Request 39330: libprocess: Replaced usage of "volatile" with std::atomic.

2015-10-18 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39330/#review103092 --- Seems like splitting the commit into two dependent commits (and

Review Request 39330: libprocess: Replaced usage of "volatile" with std::atomic.

2015-10-14 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39330/ --- Review request for mesos and Joris Van Remoortere. Bugs: MESOS-3326

Re: Review Request 39330: libprocess: Replaced usage of "volatile" with std::atomic.

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