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

Attach keys to the strategy actions #50

Open
kabasset opened this issue Jul 30, 2023 · 1 comment
Open

Attach keys to the strategy actions #50

kabasset opened this issue Jul 30, 2023 · 1 comment
Assignees
Labels
enhancement New feature or request related to an existing service question Further information is requested
Milestone

Comments

@kabasset
Copy link
Collaborator

In order to allow removing or copying actions, they should be attached a key (e.g. an encapsulated std::list::iterator, which gets not invalidated by adding or removing actions). A special value could be assigned to compression.

@kabasset kabasset added the enhancement New feature or request related to an existing service label Jul 30, 2023
@kabasset kabasset added this to the Backlog milestone Jul 30, 2023
@kabasset kabasset self-assigned this Jul 30, 2023
@kabasset kabasset added the question Further information is requested label Jul 30, 2023
@kabasset
Copy link
Collaborator Author

Example:

Strategy s;
auto id = s.append(ValidateChecksums());
s.append(Compress<HCompress>());
auto action = s[id];
s.erase(id);

Should it even be a container, i.e. with an actual iterator?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request related to an existing service question Further information is requested
Projects
None yet
Development

No branches or pull requests

1 participant