Re: boolean issue with serf

On Sat, Apr 18, 2009 at 17:43, Stefan Küng <tortoisesvn_at_gmail.com> wrote:
> Greg Stein wrote:
>> On Sat, Apr 18, 2009 at 14:07, Stefan Küng <tortoisesvn_at_gmail.com> wrote:
>>> ...
>>> I've attached a patch which replaces all if(xxx == TRUE) statements with
>>> if(xxx) (well, all that I found in trunk with a simple grep search).
>>>
>>> Not sure about the commit message though: is it really necessary to list
>>> all functions that are affected for such a change?
>>
>> I'd say "nah, don't bother."
>
> With the log message or the commit?

I was responding to your question: I don't think it is really
necessary to list all the functions.

Cheers,
-g

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1793354
Received on 2009-04-18 19:38:03 CEST