[hatari-devel] Wotanoid problem : final fix before commit ?
Eero Tamminen
eerot at users.berlios.de
Sun Jan 24 13:27:40 CET 2010
Hi,
On Sunday 24 January 2010, George Nakos wrote:
> Sunday, January 24, 2010, 1:42:02 PM, you wrote:
> > On Sunday 24 January 2010, Laurent Sallafranque wrote:
> >> >>> I've looked at the compendium.
> >> >>> I haven't found anything about this -1 length call.
> >> >>
> >> >> My memory is a bit hazy, but I think this just tells gemdos to
> >> >> read $ffffffff bytes from the file. Of course gemdos will read as
> >> >> much as the file size and then return an error in d0.l. It's the
> >> >> lazy man's approach to doing things the Right Way(tm) (i.e.
> >> >> getting the file length -or knowing it beforehand- and calling
> >> >> fread() with the proper amount of bytes)
> >
> > George, have you actually used this on actual HW so that it has worked?
> > With which TOS version(s) it works like that?
>
> Yup, on a STE with TOS 1.62UK and STFM with TOS 1.02UK. Also, that's
> we use that in ULSv3 for a couple of years now and it works in all our
> test equipment (MSTEs, Falcons, a TT, etc)
Thanks. Then it seems that EmuTOS is (as usual) broken and documentation
is wrong too & Laurent's patch is fine.
Laurent, why you apply the patch, add to code a comment about what George
mentioned above.
I guess Fwrite() size should then be Uint32 too, what do you think?
(It goes to fwrite which uses size_t, so it's converted to unsigned
anyway...)
- Eero
More information about the hatari-devel
mailing list