<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sat, Sep 6, 2014 at 5:02 PM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="">On Sat, Sep 6, 2014 at 4:38 PM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style: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></blockquote><div><br></div></span><div>The same is true for warn_cxx98_compat_enumerator_list_comma (which is in CXX98CompatPedantic) too though, right?</div></div></div></div></blockquote><div><br></div><div>…anyhoo, here's a version that just makes this warning an Extension.</div><div><br></div><div>I didn't move CXX98CompatBindToTemporaryCopy to CXX98CompatPedanticbecause that seems like an unrelated change, and because it contradicts the comment above CXX98CompatPedantic:</div><div><br></div><div><div>// Warnings for C++11 features which are Extensions in C++98 mode.</div></div><div><br></div><div>CXX98CompatBindToTemporaryCopy isn't an Extension, it's a (DefaultIgnored) Warning. If this gets moved, the comment should be updatend and the other two (CXX98CompatLocalTypeTemplateArgs, CXX98CompatUnnamedTypeTemplateArgs) should probably move too? (…but as I said, I think that's unrelated to this change.)<br></div></div><br></div></div>