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

freely mix comprehensions, spread arguments, and listed arguments #868

Open
gavinking opened this issue Dec 4, 2013 · 2 comments
Open

Comments

@gavinking
Copy link
Member

Currently, for no particularly good reason, a spread argument or comprehension must be the very last argument in a list of arguments. For #807 we should remove this limitation, allowing all three kinds of arguments to be freely mixed.

@lucaswerkmeister
Copy link
Member

I noticed that you can’t spread arguments in a comprehension:

{Result+}|NoResult ret = getResults();
{Result*} resultsOrNothing = { if (is {Result+} ret) *ret };

isn’t allowed, you have to emulate it with an extra for comprehension clause.
Would that be part of this issue?

@gavinking gavinking modified the milestones: 1.2, 1.5 Jun 21, 2014
@gavinking
Copy link
Member Author

@lucaswerkmeister that is #860.

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

No branches or pull requests

2 participants