On Wed, Jun 2, 2010 at 8:29 PM, Paweł Blokus <[email protected]> wrote: > Firstly, are the classes like TemporalIntervalYear defined somewhere? > I've made a search for them and got no hits.
No, they were to replace the mess in /drizzled/function/time/date_add_interval.cc, but I never had time to complete the work. You can safely remove those classes and the operators in TemporalInterval dealing with the interval specialization classes. > And secondly, I'm wondering about operators like > > const Date Date::operator-(const Date &rhs) > > This seems somewhat illogical to me, by subtracting two points in the > time we get another point instead of an interval. Does this really > have any applications? If we've got temporal intervals why not use > them? I'm asking mainly to know whether there is sense of writing > tests for them, but also to satisfy my curiosity :) Subtracting a date from another date produces another date, not an interval. In any case, the code isn't used AFAIK. Cheers! jay _______________________________________________ Mailing list: https://launchpad.net/~drizzle-discuss Post to : [email protected] Unsubscribe : https://launchpad.net/~drizzle-discuss More help : https://help.launchpad.net/ListHelp

