-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
%SGN {RND40} encoding bug #56
Comments
EDIT: This passes: |
Actually, just this errors: |
NealeTools
changed the title
Encoding/parsing bug in SPRITE command
%SGN {RND40} encoding bug
Apr 15, 2024
Cheers. It's the nested context that my parser is failing at. I knew I had
trouble in there just didn't have enough tests to catch it.
I'll try to get a fix in this week for that.
…On Mon, 15 Apr 2024, 22:22 MattN, ***@***.***> wrote:
Actually, just this errors:
%x=%SGN{RND40}
—
Reply to this email directly, view it on GitHub
<#56 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAADLBGL2XXFRGF5R23KPQ3Y5RAJ5AVCNFSM6AAAAABGIC646CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJXHAZTEMBQHA>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Got the code in the txt2bas core, going to release the vscode update now |
I just tried v11.7, but it still fails... |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi Remy, just trying out v1.11.6, but it failed on:
1 SPRITE %SGN {-i},%SGN {RND40-20},%SGN {RND40-20},%56,%1,,%RND2+1,%RND2+1
Results in Nonsense in BASIC. Editing the line in 2.08 (just as simple as delete the final 1 and re-add it) and it runs fine.
The text was updated successfully, but these errors were encountered: