That works (at least for me).


On 09.10.2012, at 21:59, Mariano Martinez Peck <[email protected]> wrote:

> 
> 
> On Tue, Oct 9, 2012 at 2:22 PM, Edgar J. De Cleene <[email protected]> 
> wrote:
> 
> 
> 
> On 10/9/12 4:00 AM, "Max Leske" <[email protected]> wrote:
> 
> I guess, your idea was to not use optimized encoding, right?
> But that would have meant to serialize an extra DateAndTime object. So I 
> hacked together this quick fix (see attachment) that simply encodes another 
> int32 on the stream with the offset. The materialization method took care of 
> creating a new DateAndTime object anyway.
> 
> With the fix, the Date tests pass.
> 
> Cheers,
> Max
> 
> Max:
> 
> With your changes the test still fails in Squeak.
> Very thanks for for show where the problem could be, I try see Date and 
> Timespan implementations
> 
> 
> In the meanwhile, can you test the workaround I proposed? (that is, remove 
> Date>>#fuelAccept:) 
> 
> Thanks
> 
> -- 
> Mariano
> http://marianopeck.wordpress.com
> 

Reply via email to