On Mon, Feb 25, 2013 at 8:48 PM, Ismail Pazarbasi
<[email protected]> wrote:
> 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?
Yes, this looks good to me.
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]>*/
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
