Re: svn commit: rev 1273 - trunk/subversion/libsvn_ra_dav

On Thu, Feb 14, 2002 at 10:29:56AM -0600, Karl Fogel wrote:
> kevin@tigris.org writes:
> > [...]
> > }
> > else
> > {
> > - subdir_t *subdir = apr_palloc(pool, sizeof(*subdir));
> > + if (recurse)
> > + {
> > + subdir_t *subdir = apr_palloc(pool, sizeof(*subdir));
> > [...]
>
> Any reason not to fold this into one "else if (recurse)"? Is there a
> future `else' case planned for the "if (recurse)"?
>
Nothing specific, I just generally try to have else if (conditions) only check
conditions related to the original if, for clarity's sake. I'm fine either
way.

-- 
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Kevin Pilch-Bisson                    http://www.pilch-bisson.net
     "Historically speaking, the presences of wheels in Unix
     has never precluded their reinvention." - Larry Wall
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~