----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44135/#review124898 -----------------------------------------------------------
3rdparty/libprocess/include/process/logging.hpp (line 57) <https://reviews.apache.org/r/44135/#comment187594> I wonder whether we can use something from C++11 to avoid the need for non-portable code here. From a quick skim of the reference page, `std::atomic_thread_fence(std::memory_order_seq_cst)` _might_ work, but I haven't dug in enough to say for sure. - Neil Conway On March 18, 2016, 6:49 p.m., Daniel Pravat wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44135/ > ----------------------------------------------------------- > > (Updated March 18, 2016, 6:49 p.m.) > > > Review request for mesos, Alex Naparu, Daniel Pravat, Artem Harutyunyan, Alex > Clemmer, Joris Van Remoortere, Michael Park, M Lawindi, and Yi Sun. > > > Repository: mesos > > > Description > ------- > > Libprocess: Use Windows-compatible memory fence in logging. > > > Diffs > ----- > > 3rdparty/libprocess/include/process/logging.hpp > ee680ae8ecd837f24ec7731d55680ad8440808db > > Diff: https://reviews.apache.org/r/44135/diff/ > > > Testing > ------- > > OSX: make > Windows: build/run > > > Thanks, > > Daniel Pravat > >