[phobos] datetime review

[Walter Bright]

I know you've worked hard on this, and I apologize for coming off 
sounding a bit grumpy about things.

Jonathan M Davis wrote:
>
> * datetime.core: Contains types and functions which are used by most of the 
> other modules in datetime (e.g. the enum TUnit which hold the various units of 
> time: year, hour, second, etc.).
>
> * datetime.duration: Contains the various duration types.
>
> * datetime.interval: Contains the various interval and range types.
>
>   
[...]
> * datetime.timepoint: Contains the various time point types. It also contains 
> Clock, which will give you the current time.
>
> * datetime.timezone: Contains the various time zone classes which SysTime uses.
>
>   

It bothers me that there are so many types, i.e. "various duration 
types", "various interval types", etc.

Now that I read that again, why is there both a "duration" and an 
"interval"?

Why are there multiple "various time zone classes" instead of "a" time 
zone class?

Does handling time really merit seven modules?

> The code: https://docs.google.com/leaf?id=0B-f1J0HiG2ovYmMzYWVkODMtODRiYi00NTg4LWI4MDctOTQ0MjE4YTI5Y2U1&sort=name&layout=list&num=50

Argh, I have to create Yet Another Account just to look at the code. I apologize if my comments are answered in the code source.