Re: [hpoj-devel] new "ptal-mlcd" I/O driver checked into CVS

[<pa...@rc...>]

Hi, Joe.

Joe Piolunek wrote in response to me:
> > Could you send your version of
> > sys/io.h to my non-HP address?  I'd like to diff it against what I have on
> > my system at home.
> 
> Done.
Got it.  Thanks.  It's exactly the same as the version on my 6.2 system, so
it's obviously the difference in compilers that causes the problem.

> I changed io.h to include a space between every instance of '::'. The hpoj 
> package built afterwards with no errors. I didn't want to leave the file like 
> that, so it's back to the original now. If you find a good work-around for 
> the problem, it would be great.
Thanks for the info.

> I haven't tried to install or test the package yet. I'll probably wait until 
> the io.h work-around arrives.
I just checked it into CVS.  Please do a "cvs update" and try it now.

I decided to go with the C wrapper workaround method.  It's only enabled when
necessary, to avoid a performance penalty (however small) when the workaround
isn't needed.  If you're interested, you can try running the configure script
with either the original or "fixed" version of io.h and see that configure
enables or disables the workaround as necessary.  On my system, I simulated
the problem by adding the following three lines to create a "broken" version:
#ifdef __cplusplus
#error Simulating broken C++ compiler
#endif

Thanks for your patience while I fix all of the "but it works on my system"
problems.  :-)

BTW, in case anybody has run into problems building without parallel port
support (./configure --without-par), I fixed some compilation problems in
that scenario also.

David