Dave Cramer wrote:
>
> On 26-Jul-05, at 1:23 AM, Oliver Jowett wrote:
>
>> I looked at this and the current code is certainly wrong. The timezone
>> offset of a Timestamp (deprecated method!) returns the offset of the
>> JVM's default timezone always. We should indeed be passing the target
>> calendar and using that.
>>
>> I've added that change to my patch. Interestingly none of the regression
>> tests fail with it changed; we're very short on tests that actually test
>> the with-Calendar code..
>
> Well, I actually think this little change is more of the problem than
> anything else.
I don't understand what you mean -- are you saying we shouldn't change this?
> Did you manage to cobble together a patch for me to test ?
I'll send you a current snapshot in an hour or so. I got sidetracked
into trying to work out the semantics of getDate() and getTime() on a
timestamptz, still haven't found an entirely satisfactory solution..
-O