search

remy / vscode-nextbasic

VS Code tools for NextBASIC
https://marketplace.visualstudio.com/items?itemName=remysharp.nextbasic
8 stars 2 forks source link

%SGN {RND40} encoding bug #56

Closed NealeTools closed 3 months ago

NealeTools commented 3 months ago

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.

NealeTools commented 3 months ago

EDIT: This passes: 1 SPRITE %SGN {-i},,,%56,%1,,%RND2+1,%RND2+1 So the error is in the %SGN {RND40-20} part.

NealeTools commented 3 months ago

Actually, just this errors: %x=%SGN{RND40}

remy commented 3 months ago

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 https://github.com/remy/vscode-nextbasic/issues/56#issuecomment-2057832008, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAADLBGL2XXFRGF5R23KPQ3Y5RAJ5AVCNFSM6AAAAABGIC646CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJXHAZTEMBQHA . You are receiving this because you are subscribed to this thread.Message ID: @.***>

remy commented 3 months ago

Got the code in the txt2bas core, going to release the vscode update now

NealeTools commented 3 months ago

I just tried v11.7, but it still fails... EDIT: Scratch that - I needed to reload! Thanks!