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

Re: svn commit: rev 2745 - trunk/subversion/clients/cmdline

From: Garrett Rooney <rooneg_at_electricjellyfish.net>
Date: 2002-07-29 18:47:04 CEST

On Mon, Jul 29, 2002 at 10:43:27AM -0500, Karl Fogel wrote:
> rooneg@tigris.org writes:
> > New Revision: 2745
> >
> > --- trunk/subversion/clients/cmdline/ls-cmd.c (original)
> > +++ trunk/subversion/clients/cmdline/ls-cmd.c Fri Jul 26 19:04:34 2002
> > @@ -33,28 +35,37 @@
> >
> > /*** Code. ***/
> >
> > +static int
> > +compare_cstring_as_paths (const svn_item_t *a, const svn_item_t *b)
> > +{
> > + return svn_path_compare_paths_nts ((char *)a->key, (char *)b->key);
> > +}
>
> Minor nit, Garrett: a function named "compare_cstrings_as_paths"
> implies to me that it takes cstrings directly. Do you mind if I
> change this name to something else? (Dunno what yet, but I'll figure
> out something.)

that's fine with me.

-garrett

-- 
garrett rooney                    Remember, any design flaw you're 
rooneg@electricjellyfish.net      sufficiently snide about becomes  
http://electricjellyfish.net/     a feature.       -- Dan Sugalski
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Jul 29 18:48:17 2002

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.