From: Gavin_King/Cirrus%<CI...@ci...> - 2002-08-22 05:12:42
|
Actually - I just took a closer look at the code and realised my last post was wrong.... The current functionality actually *does* copy dates + times. This occurs even though DateType, TimeType and TimestampType all extend ImmutableType. So it looks like I had intended to make the performance optimization I mentioned - but then didn't actually do it properly. Since existing applications might depend upon the current functionality I can't really do it now. *yick* I will refactor the Type heirarchy to have DateType, TimeType, TimestampType extend NullableType rather than ImmutableType to currectly reflect their actual behaviour. At least this will mean theres no gotcha to document.... |