<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, May 5, 2014 at 10:55 AM, Joerg Sonnenberger <span dir="ltr"><<a href="mailto:joerg@britannica.bec.de" target="_blank">joerg@britannica.bec.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Mon, May 05, 2014 at 05:39:12PM +0000, Logan Chien wrote:<br>
> To allow the compilation with gcc, we can't use the<br>
> __has_feature(noexcept) to detect the existence of noexcept<br>
> keyword.  We should use __cplusplus >= 201103L instead.<br>
<br>
</div>I don't think this is correct. Shouldn't it use noexcept if *either* the<br>
feature exists OR the language is C++11?</blockquote><div><br></div><div>The __cplusplus check is correct here: what we're really testing is whether we have the C++11 rule that destructors are implicitly noexcept or not (the check that 'noexcept' works is incidental). In any case, __has_feature(noexcept) will only be true if we're in C++11-or-later mode, because __has_feature looks for features that are part of the currently-enabled language mode.</div>
<div><br></div><div>=> Patch LGTM.</div></div></div></div>