On 2011-09-02 00:23, Damian Ziemba wrote:
On Thu, 01 Sep 2011 20:00:46 +0200, Jacob Carlborg wrote:
On 2011-09-01 18:28, jdrewsen wrote:
Den 01-09-2011 06:43, David Nadlinger skrev: [snip]
Jonas, do you already have a revised version ready that could
immediately be voted upon? I recognize that quite a large number of
suggestions was raised, but as there are quite a few other components
currently waiting in the review queue, we want to make sure not to
introduce any avoidable delays while working through that list.
I have done a lot of the changes but there are still quite some work
left to do. Please go on with another review and I'll get back when I'm
ready for a next round.
If not, I would propose to postpone the voting phase and start the
review for another component, e.g. David Simcha's std.regionallocator,
now. After Jonas had enough time to finish a version he considers fit
for inclusion into Phobos, I would suggest a short (one week) combined
review/vote period to decide on inclusion with the standard library.
Sounds good to me.
Thanks for running this review.
/Jonas
BTW, shouldn't etc.curl be std.net.curl now when we have a std.net
package.
In general splitting Phobos into subpackages would be great idea.
Now everything goes to root, if somebody implements more digest
mechanisms we will have in root, std.md5, std.sha256, std.sha1,
std.whirpool etc etc, std.digest.X looks way much better. Same goes with
std.datetime, it really could be splitted into std.date.time, std.date.UTC
etc etc :-D
Sorry for off-topic :-p
I agree that std.datetime should be splitted into several modules living
in a new sub package.
--
/Jacob Carlborg
