Skip to content
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

multiple assignments not working in CS7 #1801

Open
joachimheintz opened this issue Jan 1, 2024 · 6 comments
Open

multiple assignments not working in CS7 #1801

joachimheintz opened this issue Jan 1, 2024 · 6 comments
Milestone

Comments

@joachimheintz
Copy link
Contributor

this works in CS6 but not in CS7 (develop):
iMin,iMax = 5,10

not sure it is already expected to work but want to mention it for compatibility.

@kunstmusik
Copy link
Member

I was not aware of this syntax in CS6. Is this commonly used? I'm wondering about the long-term language design implications of this and whether it'll cause issues in the future.

Would love to hear @jpffitch and @vlazzarini 's thoughts on this.

@vlazzarini
Copy link
Member

I wasn't aware either. Looks like tuples to me but I never knew we supported them in the old parser.

@joachimheintz
Copy link
Contributor Author

joachimheintz commented Jan 2, 2024 via email

@vlazzarini
Copy link
Member

well then, looks like we need to keep this in the new parser. This goes back to 2010.

@kunstmusik kunstmusik added this to the 7.0.0 milestone Jan 2, 2024
@joachimheintz
Copy link
Contributor Author

joachimheintz commented Jan 10, 2024 via email

@jpffitch
Copy link
Contributor

I looked more carefilly and multiole assinment iin cs6 is just the usual call of an opcode. There is no fuss about diskin2 have multiple outputs and more than 1 input. so there is issue in =or init,. The syntax is just like elsewhere

  • { "=.i", S(ASSIGNM),0, 1, "IIIIIIIIIIIIIIIIIIIIIIII", "m",
    minit, NULL, NULL, NULL },
    { "=.k", S(ASSIGNM),0, 2, "zzzzzzzzzzzzzzzzzzzzzzzz", "z",
    NULL, minit, NULL, NULL },
    In the parser3 however = is treated specially and so needs a rewrite to match the simplicity of cs6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Ready
Development

No branches or pull requests

4 participants