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

Turn Capture into object #116

Open
phorward opened this issue Aug 7, 2023 · 1 comment
Open

Turn Capture into object #116

phorward opened this issue Aug 7, 2023 · 1 comment
Labels
discussion Further discussion on this topic required feature New feature or request

Comments

@phorward
Copy link
Member

phorward commented Aug 7, 2023

This issue was a point in the checklist on #105:

Think about a replacement of Capture by RefValue with a new value type Capture, so that Repeat returns a capture range and not a list of values

In general, this is a huge change to Tokay and the current implementation, but should be kept in mind and discussed.
Right now, a change would be to big. There's also the possibility of a hybrid-implementation. An important note is, that Capture-Objects must be referenced by the VM to turn Captures into Values as soon as the Reader is committed, and any range definitions become invalid.

@phorward phorward added feature New feature or request discussion Further discussion on this topic required tokay labels Aug 7, 2023
@phorward phorward removed the tokay label Oct 8, 2023
@phorward
Copy link
Member Author

This would maybe an alternative solution for #123 as well.

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

No branches or pull requests

1 participant