+1 for inclusion, including the unittests just the way they are.<br><br>Why? Because I've been waiting for a decent datetime module for a while and this looks like one. There's no good reason not to include the unittests. Yes, there is an abundance of them. But there's _huge_ potential for small, off-by-one bugs in something like this, and I think maximum coverage, even if some of it overlaps, can only be a good thing.<br>
<br><br><br><br><div class="gmail_quote">On Thu, Dec 9, 2010 at 8:40 PM, Jonathan M Davis <span dir="ltr"><<a href="mailto:jmdavisProg@gmx.com">jmdavisProg@gmx.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div><div></div><div class="h5">On Thursday 09 December 2010 16:26:13 Andrei Alexandrescu wrote:<br>
> Jonathan M. Davis has diligently worked on his std.datetime proposal,<br>
> and it has been through a few review cycles in this newsgroup.<br>
><br>
> It's time to vote. Please vote for or against inclusion of datetime into<br>
> Phobos, along with your reasons.<br>
<br>
</div></div>Well, I'm obviously voting for inclusion. ;)<br>
<br>
I wrote it. I'm not about to claim that it's perfect, but overall I like it and<br>
think that it's quite good. Unfortunately, either most everyone likes it and has<br>
said nothing, or they (understandably) didn't want to read it.<br>
<br>
I would point out though, that as it stands, including std.datetime would<br>
require including my time module as core.time (which has been discussed to some<br>
extent with Sean, since it was pretty much his idea in the first place that some<br>
level of integration should occur there) as well as including my unittests<br>
module as something like std.unittests (which was reviewed here on some level,<br>
and has definitely been improved from its initial version, but hasn't exactly had<br>
overwhelming support). The unittest functions could be integrated privately into<br>
std.datetime, but I think that that would be a disservice to the community at<br>
large.<br>
<br>
Regardless, I'm obviously for including it.<br>
<font color="#888888"><br>
- Jonathan M Davis<br>
</font><br>
<br>
P.S. The most recent code is here: <a href="http://is.gd/hYwOV" target="_blank">http://is.gd/hYwOV</a><br>
</blockquote></div><br>