On Sat, Sep 6, 2014 at 5:02 PM, Nico Weber <[email protected]> wrote:

> On Sat, Sep 6, 2014 at 4:38 PM, Richard Smith <[email protected]>
> wrote:
>
>> -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.
>>
>
> The same is true for warn_cxx98_compat_enumerator_list_comma (which is
> in CXX98CompatPedantic) too though, right?
>

…anyhoo, here's a version that just makes this warning an Extension.

I didn't move CXX98CompatBindToTemporaryCopy to CXX98CompatPedanticbecause
that seems like an unrelated change, and because it contradicts the comment
above CXX98CompatPedantic:

// Warnings for C++11 features which are Extensions in C++98 mode.

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.)

Attachment: clang-bind-to-temp.patch
Description: Binary data

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to