From:  Boris Zbarsky <bzbarsky@mit.edu>
Date:  18 Nov 2005 13:37:43 Hong Kong Time
Newsgroup:  news.mozilla.org/netscape.public.mozilla.layout
Subject:  

Re: Viewer.exe asserts

NNTP-Posting-Host:  adsl-69-209-222-74.dsl.chcgil.ameritech.net

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.  ;)

-Boris