[PATCH] Consolidating attribute diagnostics
Aaron Ballman
aaron at aaronballman.com
Mon Jul 22 14:45:31 PDT 2013
Can do. Assuming those changes are made, LGTY?
~Aaron
On Mon, Jul 22, 2013 at 5:39 PM, Eli Friedman <eli.friedman at gmail.com> wrote:
> On Mon, Jul 22, 2013 at 12:34 PM, Aaron Ballman <aaron at aaronballman.com> wrote:
>> This patch consolidates several attribute argument diagnostics into a
>> single, selectable diagnostic. This makes the diagnostic more
>> consistent as well. Minor functional changes caused two test cases to
>> be updated.
>
> Please use an enum for the argument types.
>
> If if isn't too inconvenient, please strip the _'s from the attribute
> names; they're useless noise.
>
> -Eli
More information about the cfe-commits
mailing list