gnupic@linuxhacker.org

Craig Franklin wrote:
> 
> Scott Dattalo wrote:
> > Instead it should be like this:
> >
> >           if ((file < 0x80) || (file > 0xf7f)) {
> >             a = 0;
> >           } else {
> >             a = 1;
> >           }
> >
> 
> I don't know how I made this mistake, but it is wrong.  I will fix it.
> 

It seemed strange that I would make an error like this.  So I reviewed
my old work.  I constructed the test file to test for the boundary at
0x60, because that is how MPASM was operating.  At the time I thought it
was strange, but I never followed up.

It looks like the boundary changes depending on what 18xx device you are
using.  My test file uses the 18f8720.  0x60 is correct for that
device.  0x80 is probably correct for the 18f452.

I need to research this some more, before any changes are made.  gpasm
assumes all 18xx devices are the same, looks like that is incorrect.