eyalroz / cuda-api-wrappers

Thin, unified, C++-flavored wrappers for the CUDA APIs
BSD 3-Clause "New" or "Revised" License
766 stars 79 forks source link

link::marshalled_options_t and link::option_t should be in namespace detail_ #622

Closed eyalroz closed 3 months ago

eyalroz commented 5 months ago

Library users never actually need to know about JIToption's (and link::marshalled_options_t), nor about the CUjit_option - since they have our nice and convenient link::options_t class, which they can populate and pass around. Their only real use is internal, and thus - they shoud be placed in the detail_ namespace.