[LLVMdev] fix warning with newer g++ compilers

Bill Wendling isanbard at gmail.com
Sun Dec 16 00:58:12 PST 2007


On Dec 15, 2007, at 12:45 PM, Dale Johannesen wrote:
> 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)
>               ;

You're ALL wrong!

   for (++CurPtr;
          isdigit(CurPtr[0]);
            ++CurPtr)
    {
     /* This space intentionally left empty */
    }

(Just kidding) ;-)

-bw



More information about the llvm-dev mailing list