> On Nov. 6, 2019, 4:03 p.m., Benjamin Bannier wrote: > > 3rdparty/stout/include/stout/attributes.hpp > > Lines 17 (patched) > > <https://reviews.apache.org/r/71734/diff/2/?file=2171706#file2171706line17> > > > > Did you want to export this macro? I am not sure it would be useful > > elsewhere since any use of it should probably be in this file. Maybe let's > > just `#undef` it once we are done here.
I wasn't sure, but it seems like a macro that could be generally useful if some code wants to test for the presence of an attribute, and it's already in the correct header for attribute-related convenience methods. (and it's namespaced, so there's no need to worry about namespace pollution) What do you think? > On Nov. 6, 2019, 4:03 p.m., Benjamin Bannier wrote: > > 3rdparty/stout/include/stout/exit.hpp > > Line 65 (original), 65 (patched) > > <https://reviews.apache.org/r/71734/diff/2/?file=2171707#file2171707line65> > > > > nit: single line? I thought about it, but right now (i.e. after this patch is applied) all code is inconsistent in using the ``` attributes void foo(args); ``` style of declaration, which looks pretty good to me. - Benno ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71734/#review218544 ----------------------------------------------------------- On Nov. 6, 2019, 3:56 p.m., Benno Evers wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71734/ > ----------------------------------------------------------- > > (Updated Nov. 6, 2019, 3:56 p.m.) > > > Review request for mesos, Benjamin Bannier, Benjamin Mahler, and Greg Mann. > > > Repository: mesos > > > Description > ------- > > This makes several changes to the attribute compatibility layer > provided by stout: > > * Add a new `STOUT_HAS_CPP_ATTRIBUTE` macro to test compiler > support for a given attribute. > * Renamed the non-namespaced `NORETURN` macro to `STOUT_NORETURN`. > * Preferred the use of the standardized `[[noreturn]]` syntax > if supported by the compiler. > * Fixed previous usages of `NORETURN` in the stout codebase. > * Added support for the `[[nodiscard]]` attribute. > > > Diffs > ----- > > 3rdparty/stout/include/stout/abort.hpp > 43ed5ce2830c493e4c801cc81f8dde0922c99a8d > 3rdparty/stout/include/stout/attributes.hpp > aa377db82e1dbdb8727b1128780e2409accc8ae9 > 3rdparty/stout/include/stout/exit.hpp > 34585a005063b17d0c7754c8e8c13f0641383bc4 > 3rdparty/stout/include/stout/unimplemented.hpp > ab6caa8fa9645bca66a3efcdc6d337f3fb0481d7 > 3rdparty/stout/include/stout/unreachable.hpp > d4b3bb0582eb9e64e6f150735d1e9f2956edbca6 > > > Diff: https://reviews.apache.org/r/71734/diff/2/ > > > Testing > ------- > > > Thanks, > > Benno Evers > >