Re: svn commit: r21710 - in branches/incomplete-directories/subversion: include libsvn_ra libsvn_ra_dav libsvn_ra_local libsvn_ra_serf libsvn_ra_svn libsvn_repos mod_dav_svn/reports svnserve tests/libsvn_repos

On 9/28/06, kfogel@tigris.org <kfogel@tigris.org> wrote:

> +/* Return svn_depth_t depth based on boolen RECURSE. This is a helper
> + for translating from the old pre-depth API. */
> +#define UNFOLD_TO_DEPTH(recurse) \
> + (recurse) ? svn_depth_infinity : svn_depth_zero

> +#define FOLD_FROM_DEPTH(depth) ((depth) == svn_depth_infinity) ? TRUE : FALSE

Tiny nit: Maybe I'm just too paranoid about macros, but I'd wrap the
right-hand side of each of these in parentheses, in case somebody
later accidentally used this in a context like "UNFOLD_TO_DEPTH(r) ==
svn_depth_infinity". Yeah, I know they're just for the compatibility
layer, but it can't hurt... (Or just make them into functions.)

(Also, s/boolen/boolean/.)

--dave

-- 
David Glasser | glasser_at_mit.edu | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org