Re: svn commit: r25554 - trunk/subversion/libsvn_wc

On Wed, Jun 27, 2007 at 06:55:20AM -0700, pburba@tigris.org wrote:
> Set TRUE/FALSE *not* non-zero/zero int when testing a WC path's switch status.
>
> * subversion/libsvn_wc/util.c
> (svn_wc__path_switched): Set *switched argument to TRUE or FALSE as per the
> docstring for this function.

In general, our doc-strings should just say something like "Set
*switched to true if ...; otherwise, set *switched to false", and not
worry about whether we're using TRUE specifically.

(and we obviously shouldn't be testing again TRUE directly).

> - *switched = strcmp(parent_child_url, entry->url);
> + *switched = strcmp(parent_child_url, entry->url) == 0 ? FALSE : TRUE;

(Incidentally, I find both of those forms really hard to parse. That
might just be me, however).

Regards,
Malcolm