From:  Boris Zbarsky <>
Date:  18 Nov 2005 13:37:43 Hong Kong Time

Re: Viewer.exe asserts


Justin Wood (Callek) wrote:
>>   if (NS_FAILED(rv) || (nsnull == cookieService)) {
>> #ifdef DEBUG
>>     printf("Unable to instantiate Cookie Manager\n");
>> #endif
>>   }
> That would save some cycles on checking those conditions.

I would expect that any modern compiler will optimize away any if block with an 
empty body if the condition has no side-effects.  Like the one here.

Please test and let me know whether I'm right, though.  ;)