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

Re: svn commit: r26419 - in trunk/subversion: include libsvn_client libsvn_wc

From: David Glasser <glasser_at_davidglasser.net>
Date: 2007-09-08 21:26:50 CEST

On 9/1/07, kfogel@tigris.org <kfogel@tigris.org> wrote:
> Author: kfogel
> Date: Sat Sep 1 14:27:45 2007
> New Revision: 26419
>
> Log:
> * subversion/include/svn_wc.h
> (svn_wc_adm_open3,
> svn_wc_adm_open2,
> svn_wc_adm_probe_try3,
> svn_wc_adm_probe_try2,
> svn_wc_adm_probe_open3,
> svn_wc_adm_probe_open2,
> svn_wc_adm_open_anchor): Change 'depth' parameter to 'levels_to_lock',
> to avoid confusion with the 'svn_depth_t' sense of "depth".
>
> Everywhere: Adjust comments and variable / static method names accordingly.

A little voice in the back of my head tells me that it's not OK to
change parameter names in public APIs, because in some bindings you
can use the parameter name as a keyword-style arg in calls. Am I just
making that up?

--dave

-- 
David Glasser | glasser_at_davidglasser.net | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Sep 8 21:23:29 2007

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.