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

Allow nested invocation for short syntax #2

Closed
Emoun opened this issue Apr 26, 2020 · 1 comment
Closed

Allow nested invocation for short syntax #2

Emoun opened this issue Apr 26, 2020 · 1 comment
Assignees
Labels
D-accepted A decision (D) has been made and the issue will be worked on I-feature This issue (I) regards a (potential) feature in the project T-accepted Triage (T): Initial review accepted issue/PR as valid

Comments

@Emoun
Copy link
Owner

Emoun commented Apr 26, 2020

Now that #1 has changed the short syntax, it should be possible to allow nested invocation in it:

#[duplicate(
  int_type  implementation;
  [i8]      [ *self < 0 ];
  #[
    int_type  ; [u8]; [u16]; [u32]
  ][ 
      [ int_type  ] [ false ]; 
  ]  
)]
@Emoun Emoun added D-accepted A decision (D) has been made and the issue will be worked on I-feature This issue (I) regards a (potential) feature in the project T-accepted Triage (T): Initial review accepted issue/PR as valid labels Apr 29, 2020
@Emoun Emoun self-assigned this Apr 29, 2020
@Emoun
Copy link
Owner Author

Emoun commented May 15, 2020

Initially, nested invocations cannot come before the list of substitution identifiers. This means, we can only really use it for producing substitution groups.

This could be changed in the future if the functionality is deemed worth it. Currently I cannot think of a good use case to being able to produce the identifier list through a nested invocation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D-accepted A decision (D) has been made and the issue will be worked on I-feature This issue (I) regards a (potential) feature in the project T-accepted Triage (T): Initial review accepted issue/PR as valid
Projects
None yet
Development

No branches or pull requests

1 participant