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

Bare sigils #126

Merged
merged 9 commits into from May 17, 2016

Conversation

Projects
None yet
2 participants
@brummett
Contributor

brummett commented May 11, 2016

This branch enables use of bare sigils as placeholders in function signatures, and is meant to resolve #100 and RT/93334

  • A bare "$" means accept any scalar value
  • A bare "@" is a synonym for "..."
  • A bare "%" is also a synonym for "...", but requires an even number of don't-care params

brummett added some commits May 9, 2016

Handle the case where the signature contains a valid $,
The only time "$," must be broken up into "$" and "," is when it's the first
thing in a signature's parameter, which can also happen if it's only seen
whitespace up to the current param's parse.  If there are any non-whitespace
tokens already seen in the current param we're working on, then this "$,"
must be a real "$,".

The only other case to consider is something like
$foo , $, , $bar
which would mean the second param is to be called "$,".  It's not possible
since you can't have a lexical var called $,

@barefootcoder barefootcoder merged commit 8ff497c into evalEmpire:master May 17, 2016

@barefootcoder

This comment has been minimized.

Show comment
Hide comment
@barefootcoder

barefootcoder May 17, 2016

Contributor

Reviewed the code and it looks excellent. Thanx so much for all this work. I will mention you in the Changes file.

Contributor

barefootcoder commented May 17, 2016

Reviewed the code and it looks excellent. Thanx so much for all this work. I will mention you in the Changes file.

@brummett brummett deleted the brummett:bare-sigils branch May 17, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment