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.

Sign up for GitHub

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

Jump to bottom

FOR EACH loops #120

Closed
Lartu opened this issue Jun 20, 2019 · 10 comments · Fixed by #121
Closed

FOR EACH loops #120

Lartu opened this issue Jun 20, 2019 · 10 comments · Fixed by #121
Labels
documentation Someone is needed in the docs! feature request Please add this to the language!

Comments

@Lartu
Copy link
Owner

Lartu commented Jun 20, 2019

It would be wonderful to have FOR EACH loops in LDPL to iterate through every element in a MAP or a LIST. Maybe something like:

FOR EACH <var of the same type of the list> IN <list> DO

REPEAT
@Lartu Lartu added the feature request Please add this to the language! label Jun 20, 2019
@Lartu Lartu added this to the LDPL 3.0.7 'Eloquent Eoraptor' milestone Jun 20, 2019
@dgarroDC
Copy link
Collaborator

I like this!

What about also using REPEAT instead of NEXT in FOR - FROM - TO - STEP - DO? I prefer if we use the same word in all loop statements.

@Lartu
Copy link
Owner Author

Lartu commented Jun 20, 2019

I'd love that! Better and simpler to understand.

@dgarroDC
Copy link
Collaborator

Done!

@Lartu
Copy link
Owner Author

Lartu commented Jun 20, 2019

Thank you very much!

@Lartu Lartu closed this as completed Jun 20, 2019
@dgarroDC
Copy link
Collaborator

Wait, this issue isn't about that, haha.

@dgarroDC dgarroDC reopened this Jun 20, 2019
@Lartu
Copy link
Owner Author

Lartu commented Jun 20, 2019

AHAHA sorry, yes, my bad.

@dgarroDC dgarroDC mentioned this issue Jun 20, 2019
Merged
@Lartu
Copy link
Owner Author

Lartu commented Jun 20, 2019

Thank you!

@Lartu
Copy link
Owner Author

Lartu commented Jun 21, 2019

@dgarroDC please don't forget to document this!

@Lartu Lartu reopened this Jun 21, 2019
@dgarroDC dgarroDC added the documentation Someone is needed in the docs! label Jun 21, 2019
@dgarroDC
Copy link
Collaborator

Done!

@Lartu
Copy link
Owner Author

Lartu commented Jun 21, 2019

Thank youuuu! <3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
documentation Someone is needed in the docs! feature request Please add this to the language!
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

FOR EACH statement dgarroDC/ldpl
2 participants
@Lartu @dgarroDC