-
Notifications
You must be signed in to change notification settings - Fork 817
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
RGBDS syntax updates #905
RGBDS syntax updates #905
Conversation
Looks good, thanks! I assume it was something like Please add |
@Rangi42 I added EDIT: nevermind I saw in discord an example. |
Oops, I forgot to get the new
|
I still think that Regex is advanced sorcery, so I'm very impressed. |
I think this is ready now. @aaaaaa123456789 @mid-kid want to review?
They're literally simpler than complete programming languages, it's just that they have a very dense syntax, and it can be hard to read which characters are regex language operators vs which are literal text. Python has a verbose regex mode that lets you space things out and add comments, like:
People have also made various alternative syntaxes that some might find easier to read/write. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Noticed a few minor things. Nice work!
This partially reverts commit 3052c6c.
Co-Authored-By: aaaaaa123456789 <aaaaaa123456789@acidch.at>
Co-Authored-By: aaaaaa123456789 <aaaaaa123456789@acidch.at>
Co-Authored-By: aaaaaa123456789 <aaaaaa123456789@acidch.at>
Updating the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Noticed a couple things that still need fixing while doing pokegold.
Implementing the new RGBDS Macro Syntax. This was done using regex. It
make compare
OK
. Although, Please review thoroughly to make sure I didn't change something I shouldn't have.Please also reference: #686