[LLVMdev] fix warning with newer g++ compilers

Dale Johannesen dalej at apple.com
Sat Dec 15 12:45:04 PST 2007


On Dec 15, 2007, at 2:13 AM, Duncan Sands wrote:

> Hi,
>
>> -    for (++CurPtr; isdigit(CurPtr[0]); ++CurPtr);
>> +    for (++CurPtr; isdigit(CurPtr[0]); ++CurPtr) ;
>
> personally I like
>
> 	for (++CurPtr; isdigit(CurPtr[0]); ++CurPtr) {}
>
> better since it is more explicit.

I think the bikeshed should be orange:

	for (++CurPtr; isdigit(CurPtr[0]); ++CurPtr)
              ;




More information about the llvm-dev mailing list