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

__PACKAGE__() as a sub, not a special token #129

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

amiri
Copy link

@amiri amiri commented Jul 14, 2013

I added PACKAGE as a sub, and I would like to check whether this is the right basic idea. It seems like to make it work as just PACKAGE, I need to make it something like selfLiteral in MoeLiterals or I need to provide some special keyword parser, like for subroutineCall in MoeProductions.

@stevan
Copy link
Member

stevan commented Jul 14, 2013

Hmm, are you sure it requires the parens? Perhaps making it a special token might not be a bad idea.

@amiri
Copy link
Author

amiri commented Jul 14, 2013

This current implementation requires parens, PACKAGE(). Otherwise the interpreter attempts to resolve it as a class. I will definitely work on making it a special token or adding a special keyword parser.

@stevan
Copy link
Member

stevan commented Jul 15, 2013

Excellent, thanks!

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

Successfully merging this pull request may close these issues.

2 participants