Re: CVS update: subversion/subversion/tests/libsvn_fs fs-test.c

On Fri, Mar 02, 2001 at 10:16:15PM -0600, Ben Collins-Sussman wrote:
> gstein@tigris.org writes:
>
> > this may be just a test program, but cutesy variable name wonkiness is
> > simply going to cause headaches.
> ...
> > stream_to_string (svn_string_t **string,
> > svn_stream_t *stream,
> > - apr_pool_t *puel)
> > + apr_pool_t *pool)
> ...
> > set_file_contents (svn_fs_root_t *root,
> > const char *path,
> > const char *contents,
> > - apr_pool_t *p00l)
> > + apr_pool_t *pool)
>
> Sorry, Mike Pilato and I changed them to silly names because we were
> trying to suppress a gcc warning: the local pool variables were
> "shadowing" the global one. We were simply following the rules about
> removing warnings. Hm, maybe we should just rename them *p or
> something. :)

Or simply toss the parameters and use the global (like all the other code
does). That's what I just did and checked in.

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/