> It can be broken up into multiple pieces.  I certainly don't object to
> that.

Sounds reasonable.

> I'm really not convinced that subclasses should be using DateTime's new()
> method.  Especially not a subclass for a calendar that has 13 months!

Well it worked because I redefined some of the subs new() is calling. :) (this didn't 
get posted to the list)

> Maybe we need to figure out what the subclasses are using and factor that
> out into a separate module which can be used by DateTime and other
> calendar modules.

I wasn't expecting you to agree to this so easily. :)  Unfortunately, I suspect it 
won't be an easy task to actually do this.

-J

--

Reply via email to