[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