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

Fix ** right-associativity, and clarify docs #1566

Merged
merged 1 commit into from
Dec 4, 2024

Conversation

Rangi42
Copy link
Contributor

@Rangi42 Rangi42 commented Dec 4, 2024

The ** exponentiation operator was supposed to be right-associative (like in Python, Ruby, Perl, Fortran, and most other languages), but it wasn't. This has been fixed from #675.

The operator precedence table was also somewhat confusing, with different precedence levels for some operators evidently just for the sake of fitting descriptions for each of them into individual cells. This has also been cleaned up:

image

@Rangi42 Rangi42 added bug Unexpected behavior / crashes; to be fixed ASAP! docs This affects the documentation (web-specific issues go to rgbds-www) rgbasm This affects RGBASM labels Dec 4, 2024
@Rangi42 Rangi42 requested a review from ISSOtm December 4, 2024 01:16
@Rangi42 Rangi42 added this to the v0.9.0 milestone Dec 4, 2024
Copy link
Member

@ISSOtm ISSOtm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@Rangi42 Rangi42 merged commit 0b147c9 into gbdev:master Dec 4, 2024
22 checks passed
@Rangi42 Rangi42 deleted the operator-associativity branch December 4, 2024 01:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Unexpected behavior / crashes; to be fixed ASAP! docs This affects the documentation (web-specific issues go to rgbds-www) rgbasm This affects RGBASM
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants