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

set, unordered_set, and deque support #28

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tuero
Copy link

@tuero tuero commented Aug 11, 2023

I'm not sure if this repo is maintained anymore, but this PR adds support for the standard library std::deque, std::set, and std::unordered_set.

I'd also like to add support for some additional types which I frequently use, and I'm wondering if this is something that would also be wanted, such as std::queue, std::priority_queue, and possibly the standard types for optional and variant. If so:

  1. std::queue and std::priority_queue do not allow for random access traversal. As such, the only way I can see an implementation for these types is to create a copy and unpeel the copy to access the internal elements. Would this be an acceptable implementation?
  2. std::optional and std::variant require C++17 support. Would this be an issue?

@tuero
Copy link
Author

tuero commented Aug 11, 2023

#27

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

Successfully merging this pull request may close these issues.

None yet

1 participant