On Wednesday, 24 August 2016 at 09:31:44 UTC, Lodovico Giaretta wrote:

I'm pleased to announce that my GSoC project, a replacement for the outdated std.xml, is now a Phobos PR! [1] It is an (almost complete) mirror of my repository [2], which is also available on DUB [3].

I would like to thank my mentor Robert burner Schadek for his great support and everybody who already gave some feedback during these months.

The PR is not meant for immediate merging. Some things still need improvement (docs/unittests/...) while others will come in a second iteration (advanced DTD handling). It is meant to for some reviews, focusing mainly on the design and usability of the library.

In the PR description you will find all the details, including a nice "wishlist" of things that I found missing in D during the development and some open questions.

So, if you have any consideration/suggestion, drop a line here or on the PR, and if you find bugs, don't hesitate to file an issue on the issue tracker of my repository.

Thank you very much!

[1] https://github.com/dlang/phobos/pull/4741
[2] https://github.com/lodo1995/experimental.xml
[3] https://code.dlang.org/packages/std-experimental-xml

I like the new cursor API very much since it enables new use cases that are were hard to do with the old API. Thanks for your hard work!

Examples are a bit scares. I like the way Phobos documents every functions with an unittest/example. I'm all for that!

Another thing I encountered some days ago was that the default error handler is using asserts to signal error. https://github.com/lodo1995/experimental.xml/issues/30 It is not obvious from the API that you're required to have a error handler.
Two ideas:
Use exceptions instead of asserts. This way it is recoverable in the default use case.
Require a error handler in the constructor.

Reply via email to