Re: Review Request 37876: stout: Replace GCC intrinsics with std::atomic.

2015-09-10 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37876/ --- (Updated Sept. 10, 2015, 6:53 p.m.) Review request for mesos, Joris Van Remoort

Re: Review Request 37876: stout: Replace GCC intrinsics with std::atomic.

2015-09-09 Thread Neil Conway
> On Sept. 9, 2015, 5:16 p.m., Joris Van Remoortere wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/os/posix/fork.hpp, line 278 > > > > > > Some places in this review chain you use the provided typedefs

Re: Review Request 37876: stout: Replace GCC intrinsics with std::atomic.

2015-09-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37876/#review98209 --- Ship it! Thanks for cleaning this all up Neil! Let's add some styl

Re: Review Request 37876: stout: Replace GCC intrinsics with std::atomic.

2015-08-28 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37876/ --- (Updated Aug. 28, 2015, 7:49 a.m.) Review request for mesos, Benjamin Hindman,

Review Request 37876: stout: Replace GCC intrinsics with std::atomic.

2015-08-28 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37876/ --- Review request for mesos. Repository: mesos Description --- MESOS-3326.