[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: Karl Fogel <kfogel_at_red-bean.com>
Date: Wed, 02 Jul 2008 17:14:04 -0400

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?

> @@ -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.

-Karl

---------------------------------------------------------------------
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:15:01 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.