<p dir="ltr">Oh, and you should also move the existing -Wc++98-compat warning to -Wc++98-compat-pedantic.</p>
<div class="gmail_quote">On 6 Sep 2014 16:38, "Richard Smith" <<a href="mailto:richard@metafoo.co.uk">richard@metafoo.co.uk</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">-Wc++98-compat doesn't make sense here: that warning group is for warning on code that is valid in the current language but not in C++98.</p>
<p dir="ltr">The right way to make this change would be to downgrade this from ExtWarn to Extension.</p>
<div class="gmail_quote">On 6 Sep 2014 16:08, "Nico Weber" <<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div>the attached (tiny) patch changes the ""C++98 requires an accessible copy constructor" warning to be DefaultIgnore, like e.g. "commas at the end of enumerator lists are incompatible with C++98" is. The reasoning is that this construct is accepted by all compilers and valid in C++11, so it doesn't seem like a useful warning to have enabled by default. Building with -pedantic, -Wbind-to-temporary-copy, or -Wc++98-compat still shows the warning.</div><div><br></div><div>The motivation is that I built re2, and this was the only warning that was emitted during the build. Both changing re2 to fix the warning and detecting clang and suppressing the warning in re2's build seem inferior than just giving the compiler a good default for this warning.</div><div><br></div><div>Ok?</div><div><br></div><div>Nico</div></div>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div>
</blockquote></div>