Lodovico Giaretta Posted in reply to LaTeigne



Permalink Reply

On Sunday, 31 July 2016 at 15:28:14 UTC, LaTeigne wrote: > On Saturday, 30 July 2016 at 09:26:27 UTC, Lodovico Giaretta wrote: >> Hi, >> >> I'm proud to announce that std.experimental.xml v0.1.0 is available on DUB [1]! >> >> This is the project I'm working on for GSoC 2016. It aims to become a substitution for Phobos std.xml. Now you can easily try it and provide some feedback. I will soon create a WIP PR on the Phobos repository. >> >> I suggest you to depend on ~master instead of v0.1.0, as bugfixes and enhancements are added on a daily basis (v0.1.0 is already one commit behind!) >> >> Current limitations: >> 1) The documentation [2] is very limited; >> 2) Some advanced DOM methods are not completely implemented; >> 3) Some advanced features (e.g. DTD validation, namespace checks) are not yet available. >> >> If you find any issue / have any suggestion, please file an issue on Github [3]. >> >> Thank you. >> >> [1] >> [2] >> [3] Hi,I'm proud to announce that std.experimental.xml v0.1.0 is available on DUB [1]!This is the project I'm working on for GSoC 2016. It aims to become a substitution for Phobos std.xml. Now you can easily try it and provide some feedback. I will soon create a WIP PR on the Phobos repository.I suggest you to depend on ~master instead of v0.1.0, as bugfixes and enhancements are added on a daily basis (v0.1.0 is already one commit behind!)Current limitations:1) The documentation [2] is very limited;2) Some advanced DOM methods are not completely implemented;3) Some advanced features (e.g. DTD validation, namespace checks) are not yet available.If you find any issue / have any suggestion, please file an issue on Github [3].Thank you.[1] https:// code.dl ang.org/ packages/ std-exper imental-xml [2] https:// lodo1995. github.io/ experime ntal.xml/ [3] https:// github.com/ lodo1995/ experim ental.xml > > I have two comments. > > What is the plan for the string interner and the allocator-based appender ? > > They are neither part of the package, nor proposed in phobos, it seems that you'll encounter a problme in the package structure itself. This is also problemtaic now if I want to test it I have to add 3 import paths to sc.conf. > > I suggest you either to propose them for phobos or to add them in a subpackage "internal" **inside xml** (or in a big internal.d module) like it's done for several phobos packages (algos, ndslices). > > _____ > > I see a naming problem in you "fast" strings: fastIndexOf, fastEqual etc. > > This is not good: does it mean that phobos's equivalent are slow ? Does it mean that you'll also propose slow equivalents (This is absurd, but it shows the problem). On Saturday, 30 July 2016 at 09:26:27 UTC, Lodovico Giaretta wrote:I have two comments.What is the plan for the string interner and the allocator-based appender ?They are neither part of the package, nor proposed in phobos, it seems that you'll encounter a problme in the package structure itself. This is also problemtaic now if I want to test it I have to add 3 import paths to sc.conf.I suggest you either to propose them for phobos or to add them in a subpackage "internal" **inside xml** (or in a big internal.d module) like it's done for several phobos packages (algos, ndslices)._____I see a naming problem in you "fast" strings: fastIndexOf, fastEqual etc.This is not good: does it mean that phobos's equivalent are slow ? Does it mean that you'll also propose slow equivalents (This is absurd, but it shows the problem).