02-14-2012, 02:30 PM
When I try to use Isqrt.bas it says:
ISqrt.bas:125: Error: illegal preprocessor character '$'
ISqrt.bas:125: Error: illegal preprocessor character '$'
Isqrt.bas (*solved*)
|
02-14-2012, 02:30 PM
When I try to use Isqrt.bas it says:
ISqrt.bas:125: Error: illegal preprocessor character '$'
02-14-2012, 11:08 PM
slenkar Wrote:When I try to use Isqrt.bas it says: Um. I wrote that piece. I'd better look. Ah. I'm calling this a compiler bug. It used to compile that. The current version doesn't. The code uses "jr nc,$+3" to mean "jump to three bytes forward from here" - $ should mean "current location", and the compiler didn't blink with this some versions ago. Here's a fix, replacing it with a label: sqrtLFsqrt16loop: sbc hl,de ; IF speed is critical, and you don't mind spending the extra bytes, ; you could unroll this loop 7 times instead of DJNZ. ; deprecated because of issues - jr nc,$+3 (note that if you unroll this loop, you'll need 7 labels for the jumps the other way!) jr nc,sqrtLFsqrthop1 add hl,de sqrtLFsqrthop1: ccf rl d rla adc hl,hl rla adc hl,hl DJNZ sqrtLFsqrt16loop However, as I noted in the comment, this is a pain if you decided to unroll the loop for maximum speed - you'd need 7 labels for the jumps! Anyway, I've updated the copy in the wiki, to make this one go away. Boriel, how did jr $+3 suddenly stop compiling?!
02-14-2012, 11:28 PM
Hmmm.
This is obviouly a bug. I need to trace back to see in which version it was introduced. :x Anyway, the fix is easy, I guess. :roll:
02-18-2012, 04:25 PM
It was another bug in the preprocessor (that changed some revisions before, to allow powerful macros).
It was a simple fix. Please, update and check, and tell me if it works:!: I see you updated iSQRT.bas in the Wiki, but now the old version should also compile too.
02-18-2012, 04:39 PM
Update: Forgot to mention, 1.2.9-s778 is available, as always, in the download page: <!-- m --><a class="postlink" href="http://www.boriel.com/wiki/en/index.php/ZX_BASIC:Archive#Latest_Development_Version">http://www.boriel.com/wiki/en/index.php ... nt_Version</a><!-- m -->
|
« Next Oldest | Next Newest »
|