Re: ZoomFloppy with OpenCBM / VICE

From: Spiro Trikaliotis <ml-cbmhackers_at_trikaliotis.net>
Date: Fri, 7 Dec 2012 12:06:25 +0100
Message-ID: <20121207110625.GB15882@trikaliotis.com>
Hello Nate,

* 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. It needs to compile to an intptr_t (64-bit
> or 32-bit addresses, depending on platform). If VICE assumes it's an
> int or does casting, that could be the problem.

Good catch.

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. I know that this has always been the
case with VICE with different libraries, in order to make VICE
self-compiling. In the long run, it might harm VICE more. This means
VICE cannot benefit from updates in the 3rd party libraries: Here of 64
bit additions of OpenCBM, in other cases (compression library) of
security bug fixes...

HTH,
Spiro

-- 
Spiro R. Trikaliotis
http://www.trikaliotis.net/

       Message was sent through the cbm-hackers mailing list
Received on 2012-12-07 12:00:06

Archive generated by hypermail 2.2.0.