On Thu, Jan 25, 2007 at 07:05:57PM -0800, hwright@tigris.org wrote:
> Modified: trunk/subversion/svn/cl.h
> +/* Print the XML header to stdout, using TAGNAME as the openning tag.
> + Use pool for temporary allocations. */
> +svn_error_t *svn_cl__xml_print_header(const char *tagname,
> + apr_pool_t *pool);
> +
> +/* Print the XML footer to stdout, using TAGNAME for the closing tag.
> + Use pool for temporary allocations. */
> +svn_error_t *svn_cl__xml_print_footer(const char *tagname,
> + apr_pool_t *pool);
> +
I think we might be a bit clearer about what the 'header' and 'footer'
represent, in this case, the XML prolog and document root element
start-tag, and the root element end-tag respectively.
> Modified: trunk/subversion/svn/util.c
> +svn_error_t *
> +svn_cl__xml_print_header(const char *tagname,
> + apr_pool_t *pool)
> +{
> + svn_stringbuf_t *sb = svn_stringbuf_create("", pool);
> +
> + /* <?xml version="1.0" encoding="utf-8"?> */
> + svn_xml_make_header(&sb, pool);
I know you just copied that comment, but it's actually been out of date
since r18931.
Regards,
Malcolm
- application/pgp-signature attachment: stored
Received on Fri Jan 26 15:03:56 2007