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

Re: svn commit: r31978 - trunk/subversion/svnlook

From: Julian Foad <julianfoad_at_btopenworld.com>
Date: Wed, 02 Jul 2008 22:54:12 +0100

On Wed, 2008-07-02 at 17:14 -0400, Karl Fogel wrote:
> julianfoad_at_tigris.org writes:
> > --- trunk/subversion/svnlook/main.c (r31977)
> > +++ trunk/subversion/svnlook/main.c (r31978)
> > @@ -1533,6 +1540,48 @@ do_pget(svnlook_ctxt_t *c,
> > }
> >
> >
> > +/* A copy of svn_cl__print_xml_prop(). */
> > +static void
> > +print_xml_prop(svn_stringbuf_t **outstr,
> > + const char* propname,
> > + svn_string_t *propval,
> > + apr_pool_t *pool)
>
> My alarm bells went off when I saw that, naturally :-).
>
> Why not just use svn_cl__print_xml_prop(), then?

It's not a library function, it's local to the "svn" client. I didn't
think it was quite worth moving it into a library for this amount of
duplication. Want me to? (Uh, libsvn_client/cmdline.c ?)

> > @@ -1542,6 +1591,7 @@ static svn_error_t *
> > do_plist(svnlook_ctxt_t *c,
> > const char *path,
> > svn_boolean_t verbose,
> > + svn_boolean_t xml,
> > apr_pool_t *pool)
>
> The do_plist() function has a doc string; it should be updated for the
> new 'xml' parameter.

Oops. r31979.

Thanks.
- Julian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-07-02 23:54:44 CEST

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.