-
Notifications
You must be signed in to change notification settings - Fork 36
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
group should be more idempotent #120
Comments
And why not Cat ( Char 'x' ) ( Union ( Char ' ' ) Line ) ? |
Regarding the ideas to move the |
I think that within |
So the difficulty here are cases like In my testing (in the kotlin version) I have two methods, one I am unsure on how this affects performance because I have no good benchmarks yet ^^ |
What's I had previously also thought that we could get rid of the I'm looking forward to your experiment in any case – even if it doesn't work out, we'll know more and can document it! :) |
My bad! Came from the kotlin version where I needed to rename |
The function wrappers |
Currently we have:
It would be nice if we could get just
instead. This patch should do the trick:
But maybe we could even get this?!
Related: #99, #112, #115.
The text was updated successfully, but these errors were encountered: