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

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

From: Daniel Rall <dlr_at_finemaltcoding.com>
Date: 2005-10-20 21:03:00 CEST

On Thu, 20 Oct 2005, Ivan Zhakov wrote:

> On 10/20/05, Daniel Rall <dlr@finemaltcoding.com> wrote:
> > On Thu, 20 Oct 2005, zhakov@tigris.org wrote:
...
> > > --- trunk/subversion/libsvn_wc/props.c (original)
> > > +++ trunk/subversion/libsvn_wc/props.c Thu Oct 20 10:34:03 2005
> > > @@ -208,6 +208,44 @@
> > >
> > > /*** Misc ***/
> > >
> > > +/* Opens reject temporary file for FULL_PATH. */
> > > +static svn_error_t *
> > > +open_reject_tmp_file (apr_file_t **fp, const char **reject_tmp_path,
> > > + const char *full_path,
> > > + svn_wc_adm_access_t *adm_access,
> > > + svn_boolean_t is_dir, apr_pool_t *pool)
> > > +{
> > > + const char *tmppath, *tmpname;
> >
> > There is a parameter named "reject_tmp_path" to this function, yet neither
> > of the local variables defined above uses that same -- slightly more
> > comprehensible -- naming convention which uses the underscore to emphasize
> > word boundries (e.g. "tmp_path"). Both style came from pre-refactoring, but
> > it would be nice if the refactoring made them consistent.
>
> To be clear: do you propose simply rename tmppath and tmpname
> variables to tmp_path and tmp_name and nothing more? I don't take
> myslef rename it on refactoring, because I am nott familiar with
> subversion naming conventions in edge cases.

Yeah, that's what I am suggesting.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Oct 20 21:02:58 2005

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.