Kelvinyu1117 commented on issue #502: URL: https://github.com/apache/datasketches-cpp/issues/502#issuecomment-5702499302
I think option A makes more sense to me, as the modules are grouped together. Agree on @leerho suggestion that we should use namespaces deliberately to avoid name collisions. Also, if we want to move a bit further, like C++ 20, we can use `concept` to add a bit more type safety to the template argument I am definitely willing to help with this migration. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
