LGTM

On Fri, Sep 13, 2013 at 11:26 AM, Reid Kleckner <[email protected]> wrote:

> ================
> Comment at: lib/Lex/Pragma.cpp:1106
> @@ +1105,3 @@
> +        // Collect the warning ids.
> +        SmallVector<int, 1> Ids;
> +        PP.Lex(Tok);
> ----------------
> Richard Smith wrote:
> > Only 1 inline ID? Cheapskate! =) Maybe 10? 16?
> These are usually issued one by one.  :)


IIRC I've written code which had about 5 or 6 warning numbers listed in a
single pragma. But whatever you prefer, this is never going to matter in
practice.
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to