----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36845/#review93230 -----------------------------------------------------------
Ship it! 3rdparty/libprocess/3rdparty/stout/include/stout/thread_local.hpp (line 20) <https://reviews.apache.org/r/36845/#comment147553> micro-nit #1: Is there a link explaining this that is not behind the registration wall? 3rdparty/libprocess/3rdparty/stout/include/stout/thread_local.hpp (line 22) <https://reviews.apache.org/r/36845/#comment147554> micro nit #2: either s/required/require/, or point to where the requirement was made. - Artem Harutyunyan On July 27, 2015, 2:05 p.m., Joris Van Remoortere wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36845/ > ----------------------------------------------------------- > > (Updated July 27, 2015, 2:05 p.m.) > > > Review request for mesos, Benjamin Hindman, Artem Harutyunyan, and Michael > Park. > > > Bugs: MESOS-3119 > https://issues.apache.org/jira/browse/MESOS-3119 > > > Repository: mesos > > > Description > ------- > > This replaced the ThreadLocal primitive with the new c++11 standard. > The exception is on OSX where we use `__thread` as thread_local is not > supported. > > > Diffs > ----- > > 3rdparty/libprocess/3rdparty/stout/Makefile.am > 89e7b1854bd7f449f4f0027d76c6430d259a24de > 3rdparty/libprocess/3rdparty/stout/include/Makefile.am > 5c19e3ef8ba50ab007eda26b752441f076ca7ed0 > 3rdparty/libprocess/3rdparty/stout/include/stout/thread.hpp > 552d6e97c882a36d6a889af205c422e51f544b34 > 3rdparty/libprocess/3rdparty/stout/include/stout/thread_local.hpp > PRE-CREATION > 3rdparty/libprocess/3rdparty/stout/tests/thread_tests.cpp > 319fcdf517b24f5bb9c85dad4093b09ec87e915e > > Diff: https://reviews.apache.org/r/36845/diff/ > > > Testing > ------- > > dependens on follow-up patch. > > > Thanks, > > Joris Van Remoortere > >
