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
Reject reserved words #613
Conversation
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.
Thanks!!
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.
Niceeeeeeee 🤩
Big renames
(CCodeGenerator reservedWords includes: | ||
aString) ifTrue: [ | ||
TranslationError signal: aString , ' is a reserved word' ]. |
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.
For me, this is polémico. Do we want this validation in the setter? I was thinking to have this validation before the next transformation. But I don't know how it is used...
Also, we can have a validateReservedWords
method as in TMethod
.
Tests passing, ready to merge |
Thanks @ivojawer !! |
fixes #429
CogSimStackEntry
Selector + accessors namedregister
toregisterr