You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by "Peter N. Lundblad" <pe...@famlundblad.se> on 2006/03/28 19:20:43 UTC
Re: svn commit: r19063 - trunk/subversion/libsvn_ra_serf
jerenkrantz@tigris.org writes:
> Modified: trunk/subversion/libsvn_ra_serf/ra_serf.h
> URL: http://svn.collab.net/viewvc/svn/trunk/subversion/libsvn_ra_serf/ra_serf.h?pathrev=19063&r1=19062&r2=19063
> ==============================================================================
> --- trunk/subversion/libsvn_ra_serf/ra_serf.h (original)
> +++ trunk/subversion/libsvn_ra_serf/ra_serf.h Tue Mar 28 10:11:50 2006
> @@ -328,6 +328,22 @@
> serf_request_t*
> svn_ra_serf__request_create(svn_ra_serf__handler_t *handler);
>
> +/* XML helper callbacks. */
> +
> +typedef svn_error_t *
> +(*svn_ra_serf__xml_start_element_t)(void *baton,
> + const char *name,
> + const char **attrs);
> +
Please don't forget to document the semantics of these:-) Maybe
namespace handling could be pulled to this level as well?
> +typedef svn_error_t *
> +(*svn_ra_serf__xml_cdata_element_t)(void *baton,
> + const char *data,
> + apr_size_t len);
> +
There's no such thing as a "CDATA element". I suggest calling this
..._cdata_chunk to emphasize the fact that character data may be split
> arbitrarily
by the parser.
Thanks,
//Peter
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Re: svn commit: r19063 - trunk/subversion/libsvn_ra_serf
Posted by "Peter N. Lundblad" <pe...@famlundblad.se>.
Justin Erenkrantz writes:
> > Maybe
> > namespace handling could be pulled to this level as well?
>
> Yes, that's a thought. It depends how much we want to diverge from
> the expat callback prototype.
AFAICT, newer expat has namespace support. We could follow that pattern.
//Peter
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Re: svn commit: r19063 - trunk/subversion/libsvn_ra_serf
Posted by Justin Erenkrantz <ju...@erenkrantz.com>.
On 3/28/06, Peter N. Lundblad <pe...@famlundblad.se> wrote:
> Please don't forget to document the semantics of these:-)
Sure, sure. =)
> Maybe
> namespace handling could be pulled to this level as well?
Yes, that's a thought. It depends how much we want to diverge from
the expat callback prototype.
> > +typedef svn_error_t *
> > +(*svn_ra_serf__xml_cdata_element_t)(void *baton,
> > + const char *data,
> > + apr_size_t len);
> > +
> There's no such thing as a "CDATA element". I suggest calling this
> ..._cdata_chunk to emphasize the fact that character data may be split
> arbitrarily by the parser.
Sure. That works. r19066. -- justin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org