[cfe-commits] [PATCH] Add Fixit for C++11 attributes that appertain to class specifiers.

[Michael Han]

Hi Dmitri,

Thanks! Attach updated patch.

I noticed that not all test cases under clang/test/FixIt use
"-fdiagnostics-parseable-fixits". Is there a reason why not all FixIt test
cases use this option?

Cheers
Michael

On Sat, Dec 1, 2012 at 5:21 AM, Dmitri Gribenko <gribozavr at gmail.com> wrote:

> On Sat, Dec 1, 2012 at 6:30 AM, Michael Han <fragmentshaders at gmail.com>
> wrote:
> > This patch adds two FixIts to parser when parsing C++11 attributes
> appertain
> > to class specifier. It is a following up patch of a previous patch [1].
> >
> > Please review, thanks!
>
> Hello Michael,
>
> FixIts can be tested with -fdiagnostics-parseable-fixits (search the
> testsuite for this option to find examples).
>
> Dmitri
>
> --
> main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
> (j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20121201/58abd581/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fixit.patch
Type: application/octet-stream
Size: 7594 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20121201/58abd581/attachment.obj>