Re: Re: xml_parser_create patch
| From: | Adam Dickmeiss | Date: | Wed, 07 May 2003 10:59:54 +0000 |
| Subject: | Re: Re: xml_parser_create patch | ||
| References: | 1 2 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-1341@lists.php.net to get a copy of this message | ||
On Wed, May 07, 2003 at 12:33:20PM +0200, nicos@php.net wrote:
> -1 for the patch if it doesn't have a default value for the encoding.
>
> We don't want to break BC again.
I respect that. I hope Streling does too with a new layer on
top ..
How about letting the empty string mean : Auto encoding??
xml_parser_create(""); // any decoding! (pass 0 to Expat)
xml_parser_create(); // use default for input&output
xml_parser_create("x"); // Use decoding x for input&output
-- Adam
> --
> Regards.
> M.CHAILLAN Nicolas
> nicos@php.net
> www.WorldAKT.com Hébergement de sites internets.
> "Adam Dickmeiss" <adam@indexdata.dk> a écrit dans le message news:
> 20030507094303.GA7028@indexdata.com...
> > Hi,
> >
> > I'd like to submit a small patch that allows xml_parser_create
> > and xml_parser_create_ns (Expat version) to honor the encoding of
> > an XML document (including UTF-16). This solves bug #23293. With this
> > patch xml_parser_create now works identical to the libxml2 version of it
> > (AFAIK).
> >
> > Note that this patch _may_ break applications that assumes
> > ISO-8859-1 (or default encoding) for XML applications, but only
> > if the application omitted a desired encoding for xml_parser_create.
> >
> > Applications that wish their XML parser to do decoding
> > but want a specific target encoding can use option
> > XML_OPTION_TARGET_ENCODING .
> >
> > -- Adam
> >
> > --
> > Adam Dickmeiss mailto:adam@indexdata.dk
> > http://www.indexdata.dk
> > Index Data T: +45 33410100 Mob.: 212 212 66
> >
>
>
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
--
Adam Dickmeiss mailto:adam@indexdata.dk http://www.indexdata.dk
Index Data T: +45 33410100 Mob.: 212 212 66