On Sunday, 31 July 2016 at 15:36:47 UTC, Lodovico Giaretta wrote:
On Sunday, 31 July 2016 at 15:28:14 UTC, LaTeigne wrote:
On Saturday, 30 July 2016 at 09:26:27 UTC, Lodovico Giaretta
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.
Yes that's the most reasonable solution, unless someone has the
time to polish it for you and enough karma to get it pulled in
the allocator package (very unlikely as adding new stuffs in
phobos is usually not a piece of cake. Anyway it would really
have its place there since there's already all the array
routines: make expand shrink etc.)
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
Ok, I'm gonna check this tomorrow.
