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

Accept vectors of std::strings for setting NVRTC definition terms and values #403

Closed
eyalroz opened this issue Sep 9, 2022 · 1 comment

Comments

@eyalroz
Copy link
Owner

eyalroz commented Sep 9, 2022

We already have multiple variants of set_headers() and add_headers() implemented. However - those templated variants which take containers or container pairs assume the element type is a constant C-style string of constant characters (const char* const). What if the user has vectors of std::strings? It's a legitimate scenario... let' cater to it.

eyalroz added a commit that referenced this issue Sep 10, 2022
* Added ability to add a single named header
* Reduced the number of `add_headers()` and `set_headers()` function variants, so that the user calls a templated version and the type is determined autogneioammtl
* Moved some add_headers() calls into a proptecte name=
eyalroz added a commit that referenced this issue Sep 10, 2022
* Added ability to add a single named header to a program
* Reduced the number of `add_headers()` and `set_headers()` member variants, so that the user calls a templated version and the type is determined by the user's choice of arguments
* Made some `add_headers()` members protected, and suffixed with an `_` - the public variants call them
eyalroz added a commit that referenced this issue Sep 10, 2022
* Added ability to add a single named header to a program
* Reduced the number of `add_headers()` and `set_headers()` member variants, so that the user calls a templated version and the type is determined by the user's choice of arguments
* Made the lowest building-blocks of header-adding protected members, which the public variants call - for easier handling of types and catching of inappropriate argument type errors
@eyalroz

This comment was marked as outdated.

@eyalroz eyalroz self-assigned this Sep 10, 2022
eyalroz added a commit that referenced this issue Sep 10, 2022
* Added ability to add a single named header to a program
* Reduced the number of `add_headers()` and `set_headers()` member variants, so that the user calls a templated version and the type is determined by the user's choice of arguments
* Made the lowest building-blocks of header-adding protected members, which the public variants call - for easier handling of types and catching of inappropriate argument type errors
@eyalroz eyalroz closed this as completed in d783f98 Oct 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant