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

WIP: Use simpler syntax in grammar names #226

Closed
wants to merge 1 commit into from
Closed

Conversation

Mouq
Copy link
Contributor

@Mouq Mouq commented Oct 30, 2013

Things like token package_declarator:sym<grammar> become token package_declarator:grammar

The only nits here are that the line
<?sigil> <var=.LANG('MAIN', 'term:variable')>
which seems like it should work, doesn't in NQP, and must be:
<?sigil> <var=.LANG('MAIN', 'term:variable<1>')>
and that we can't use
method package_declarator:grammar($/)
but instead
method package_declarator:grammar ($/)
because NQP (and Perl) will think ($/) is the value of the :grammar colonpair

@Mouq
Copy link
Contributor Author

Mouq commented Oct 30, 2013

This causes some failures in spectest. Consider this PR a WIP – it's current purpose is to point out the possibility, ask for the possibility's consideration, and do most of the work implementing it

@Mouq
Copy link
Contributor Author

Mouq commented Jan 17, 2014

Honestly, this is unnecessary and purely cosmetic, other than causing some issues with name look-up. Closing for lack of usefulness

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.

1 participant