std.experimental.xml available on DUB

[Lodovico Giaretta via Digitalmars-d-announce]

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] https://code.dlang.org/packages/std-experimental-xml
>> [2] https://lodo1995.github.io/experimental.xml/
>> [3] https://github.com/lodo1995/experimental.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).

Thank you for your comments.

Talking about your points:
1) the interner shall really go away before inclusion in Phobos; 
it is unneeded; its code is already partially duplicated in 
CopyingCursor (std.experimental.xml.cursor); but it would be good 
to have something like this in Phobos, somewhere in the future.
2) The appender is needed, as the Phobos one does not work with 
custom allocators; I don't have the time to polish it for Phobos 
adoption, so putting it in an internal xml submodule may be a 
great idea.
3) The fastXXX functions are intended for internal usage; they 
will have package protection in the final library (I really 
forgot about this thing; thanks).

I will tag v0.1.1 late this night, with some fixes based on the 
feedback from you and Steven.

Thank you again.