Re: [libxml++] RE: [xml] xmlCreatePushParserCtxt() docs wrong (wa s RE: [libxml++ ] sax_pars er example: entity not defined)

[Daniel Veillard <veillard redhat com>]

On Wed, Nov 26, 2003 at 10:07:17AM +0100, Murray Cumming Comneon com wrote:
> > Well, maybe you want it to be NULL to "make everything work" in your case,
> but from an libxml2 API it's a breakage, and need to be fixed. Simply force
> it > to NULL in your wrappers, the semantic should not change even if the
> bug get fixed.
> 
> Your documentation for the SaxHandler callbacks
>  http://www.xmlsoft.org/html/libxml-parser.html#commentSAXFunc
> says that the SAX callbacks receive a context, so I don't see how you can
> sensible fix the
> xmlCreatePushParserCtxt function 
>  http://xmlsoft.org/html/libxml-parser.html#xmlCreatePushParserCtxt
> so that it sends something else instead. And I don't see the need, because
> people can just use the private_ field of the context.

  yeah, I know. There is some redundancy because there have been some
breakage of the normal way.

> Just trying to be helpful.

  Not denying it :-), I'm just trying to avoid surprise and break client
code. Maybe the simplest is to burry the SAX API and not touch it now
that the SAX2 version works reasonably well...

Daniel

-- 
Daniel Veillard      | Red Hat Network https://rhn.redhat.com/
veillard redhat com  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/