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

[BUG]: Remove all wrong usages of cuda:: and std:: namespace in CUB #3734

Closed
1 task done
fbusato opened this issue Feb 7, 2025 · 1 comment · Fixed by #3751
Closed
1 task done

[BUG]: Remove all wrong usages of cuda:: and std:: namespace in CUB #3734

fbusato opened this issue Feb 7, 2025 · 1 comment · Fixed by #3751
Assignees
Labels
feature request New feature or request.

Comments

@fbusato
Copy link
Contributor

fbusato commented Feb 7, 2025

Is this a duplicate?

Area

CUB

Is your feature request related to a problem? Please describe.

  • cuda::std is used without the full namespace ::cuda::std in several places
  • std:: is used instead of ::cuda::std

Describe the solution you'd like

fix the namespaces

Describe alternatives you've considered

No response

Additional context

No response

@fbusato fbusato added the feature request New feature or request. label Feb 7, 2025
@fbusato fbusato self-assigned this Feb 7, 2025
@fbusato fbusato added this to CCCL Feb 7, 2025
@github-project-automation github-project-automation bot moved this to Todo in CCCL Feb 7, 2025
@bernhardmgruber
Copy link
Contributor

Mind that there are some uses of std:: that are justified. std::iterator_traits is one example.

@fbusato fbusato moved this from Todo to In Progress in CCCL Feb 7, 2025
@cccl-authenticator-app cccl-authenticator-app bot moved this from In Progress to In Review in CCCL Feb 7, 2025
@github-project-automation github-project-automation bot moved this from In Review to Done in CCCL Feb 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request.
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants