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

Multiple ring methods in same line #32

Open
GuillemGongora opened this issue May 4, 2020 · 2 comments
Open

Multiple ring methods in same line #32

GuillemGongora opened this issue May 4, 2020 · 2 comments
Labels
enhancement New feature or request

Comments

@GuillemGongora
Copy link

GuillemGongora commented May 4, 2020

It could be useful to save space to be able to choose transformational ring methods to be called right after generative methods. For example:
ring notes spreadInclusive(4 0 3)
ring notes palin(notes)
to
ring notes spreadInclusive(4 0 3) palin(notes)
or something similar.
There might be a better option to call for the spreadInclusive() generated ring in the palin().

@tmhglnd
Copy link
Owner

tmhglnd commented May 4, 2020

Yeah totally agree! I'm actually working towards a parser where you can have nested functions, but this may take a bit longer. Maybe I can implement this in the meantime if it is easier.

Otherwise it will look something like this in the future

ring notes palin(spreadInclusive(4 0 3))

@tmhglnd tmhglnd added the enhancement New feature or request label May 4, 2020
@tmhglnd
Copy link
Owner

tmhglnd commented Dec 2, 2020

There is currently a small update on this. I included the & character as a way to add multiple lines on one line. For example:

ring notes spreadInclusive(5 0 12)
ring notes palindrome(notes)

Can now also be written as:

ring notes spreadInclusive(5 0 12) & ring notes palindrome(notes)

I'm thinking about a way where you don't need to type ring notes twice and the parser can see that, if the first line starts with ring notes, it will reuse that keyword for the second line as well. Which goes back to your original idea:

ring notes spreadInclusive(5 0 12) & palindrome(notes)

But this is a bit more complicated and not implemented yet.

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

No branches or pull requests

2 participants