Re: ZoomFloppy with OpenCBM / VICE

On Dec 9, 2012, at 11:06 PM, Wolfgang Moser wrote:

> Hi Spiro, Nate,
> 
> Spiro Trikaliotis schrieb:
>> * On Thu, Dec 06, 2012 at 11:30:55AM -0800 Nate Lawson wrote:
>> 
>>> Sorry to follow up so quickly, but it could also be a 64-bit problem.
>>> The area reserved for the OpenCBM CBM_FILE HANDLE object may not be
>>> treated properly by VICE. [...]
>> 
>> VICE includes the opencbm.h file directly in the source distribution of
>> VICE. Unfortunately, that file there is old, and it does not handle 64
>> bit platforms.
>> 
>> Replacing it (src/opencbm.h) through the opencbm.h file from the git
>> head of OpenCBM (opencbm/include/opencbm.h) might fix the issue.
>> 
>> Of course, it is arguable if opencbm.h should be in the VICE source
>> distribution in the first place.
>> [...]
> 
> do you both remember that once we had a discussion on this topic. One
> possible goal of three discussed solution was to manage 64-bit
> compatibility of "CBM_FILE" internally to OpenCBM, so that "CBM_FILE" is
> always a 32-bit integer, even under 64-bit systems.
> 
> This would manage binary compatibility to older 0.4.x OpenCBM releases.
> This VICE incompatbility is probably just one example of this issue.
> 
> Oh, and sorry for pointing this out while having failed to give an
> implementation for that internal mapping so far ;-)


I don't really think it's worth it since this was one ABI change in many years and isn't likely to happen often. 

Maintaining an ABI is much harder than hiding handle sizes. You also have to keep old function signatures around.

-Nate


       Message was sent through the cbm-hackers mailing list