Re: PHP binding for LibYAML
| From: | Bryan Davis | Date: | Fri, 27 Nov 2009 20:10:19 +0000 |
| Subject: | Re: PHP binding for LibYAML | ||
| References: | 1 2 | Groups: | php.pecl.dev |
| Request: | Send a blank email to pecl-dev+get-6852@lists.php.net to get a copy of this message | ||
Lukas Kahwe Smith wrote:
> Looking at how YAML is used in most PHP projects I have seen, they all
> require merge key support [1], which is not part of the core standard
> and is therefore not included in the previous YAML extension. Would be
> awesome to one day get support for merge key in a C extension.
Merge support is high on my TODO list. The parser implementation is
largely rsky's original code and it will need quite a bit of reworking
to make merge work correctly. LibYAML uses a SAX-like event notification
model and the current code discards too much information about mapping
keys to reliably detect if a key matching "<<" was unquoted or not
> Also IIRC the current extension didn't really do a good job of
> "guessing" where a potential syntax error occurred. Would be great
> if your extension could solve those two points.
Rsky's code generically reported "parser error" on any parsing failure.
I added code to pass along the line and character locations where the
library detected an error via a warning event.
Bryan
--
The moral is that coding standards are so bad that security flaws are
this common. -- Bruce Schneier
Attachment: [application/x-pkcs7-signature] S/MIME Cryptographic Signature smime.p7s
Attachment: [application/x-pkcs7-signature] S/MIME Cryptographic Signature smime.p7s