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

[FEA] Use sentinel namespace in all applicable map types #162

Closed
PointKernel opened this issue May 25, 2022 · 1 comment · Fixed by #163
Closed

[FEA] Use sentinel namespace in all applicable map types #162

PointKernel opened this issue May 25, 2022 · 1 comment · Fixed by #163
Labels
good first issue Good for newcomers type: feature request New feature request

Comments

@PointKernel
Copy link
Member

Is your feature request related to a problem? Please describe.
sentinel namespace has been added in #142 , which provides a wrapper around sentinel values. Currently, this namespace is only applied in cuco::static_map.

Describe the solution you'd like
Use sentinel namespace consistently across all applicable map types.
@PointKernel PointKernel added type: feature request New feature request good first issue Good for newcomers labels May 25, 2022
@m3g4d1v3r
Copy link
Contributor

Hello PointKernel!
Is #163 a good approach for this issue? Maybe I'm forgetting something...

@PointKernel PointKernel mentioned this issue May 26, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers type: feature request New feature request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Apply sentinel namespace to other map structures m3g4d1v3r/cuCollections
2 participants
@PointKernel @m3g4d1v3r