[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r35631 - in trunk/subversion: libsvn_client libsvn_fs_base libsvn_fs_base/bdb libsvn_fs_base/util libsvn_fs_fs libsvn_ra_local libsvn_ra_neon libsvn_ra_serf libsvn_ra_svn libsvn_repos libsvn_subr libsvn_wc mod_dav_svn svn svnlook svnserve tests/libsvn_subr

From: Blair Zajac <blair_at_orcaware.com>
Date: Mon, 02 Feb 2009 08:05:14 -0800

Julian Foad wrote:
> Author: julianfoad
> Date: Mon Feb 2 07:56:44 2009
> New Revision: 35631
>
> Log:
> Replace the expressions "? TRUE : FALSE" and "? FALSE : TRUE", which are ugly
> and redundant or almost redundant, with more direct ways to express the
> boolean test that is meant, throughout the C code base. No functional change.
>
> The changes are similar to these:
> boolean ? TRUE : FALSE => boolean
> boolean ? FALSE : TRUE => ! (boolean)
> x == y ? TRUE : FALSE => x != y

I haven't reviewed all the changes, but for the third one you mean

x == y ? FALSE : TRUE => x != y

Are there
Blair

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1091233
Received on 2009-02-02 17:05:43 CET

This is an archived mail posted to the Subversion Dev mailing list.