[IronPython] array.fromfile

[JoeSox]

Outstanding news! It worked.  I finally had some extra time to change
the other two methods to match my modifications to CreateData and it
worked like a charm. Pretty cool.  I think this is the best way to go,
so not too many people get broken.

Thanks, JoeSox


On 2/24/07, Dino Viehland <dinov at exchange.microsoft.com> wrote:
> Unfortunately there are several switch statements that will need to be updated.  They are in CreateData (generic type), ToStream (cast), FromStream (call Read?Int32).  If you update all 3 you should be consistent then.
>
> If I had to take a guess here I would think maybe there's an earlier read from an array that's driving the # of elements to be read later.
>
> I'm not opposed to changing the sizes of these to match CPython but I suspect we can never make the sizes 100% compatible across all architectures on all platforms.  Technically this IS a bug in the module you're using - it shouldn't expect the sizes to be fixed.  We can make these 4 byte values but I suspect we'll have differences on x64.  We can make them platform dependent but not all platforms have the same definitions for the size of "long".  In particular I believe Windows defines this as being 4 bytes and most other platforms choose 8 bytes.  Based upon that investigating I'll update the sizes so we can be more compatible.  But I'd love to hear if updating all 3 fixes your problem.
> ________________________________________