-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[VecOps] Make free functions in VecOps better visible #7699
Comments
Reopening, we have to check that the PR behaves as expected. |
@eguiraud What do you thinks, good like that? -> https://root.cern/doc/master/group__vecops.html |
I've no idea how to make the preproc macros look nicer or even expand them properly :/ |
We can either "expand" the macros manually (i.e. get rid of the macros), or maybe better we can explicitly write doxygen strings for them using the |
Expanding the macros would result in a massive code duplication, see https://github.com/root-project/root/blob/master/math/vecops/inc/ROOT/RVec.hxx#L607-L695 So adding manual docs would be the solution, i guess! |
Fixed by #15277 |
\defgroup
section.The text was updated successfully, but these errors were encountered: