On Tue, Feb 19, 2013 at 1:11 AM, Ismail Pazarbasi <[email protected]> wrote: > On Tue, Feb 19, 2013 at 12:43 AM, Dmitri Gribenko <[email protected]> wrote: >> On Tue, Feb 19, 2013 at 1:02 AM, Ismail Pazarbasi >> <[email protected]> wrote: >> <0001-Fix-for-Bug-7256-clang-should-provide-a-fix-it-hint-.patch> >> >> + if (const CXXRecordDecl* Class = dyn_cast_or_null<CXXRecordDecl>(Ctx)) >> >> LLVM style is to put space before the star. >> >> 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 <[email protected]>*/ > > I have fixed the whitespace issue. > > Could you please check whether the patch for the test is correct? I > have added a line of RUN command/instruction with > -fdiagnostics-parseable-fixits based on the one in test/FixIt/typo.c. > I hope I've got that right. > > Thanks!
Ping! Does this look good? _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
