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
[RF] Avoid code duplication with new private Algorithms.h
file
#11962
Conversation
The `RooMomentMorphND` and `RooMomentMorphFuncND` classes duplicated some copy-pasted code from stackoverflow. This is not factored out into a new private header file to avoid code duplication.
Starting build on |
Build failed on ROOT-ubuntu18.04/nortcxxmod. Warnings:
And 60 more Failing tests:
|
Build failed on ROOT-performance-centos8-multicore/cxx17. Warnings:
And 130 more Failing tests:
|
Build failed on ROOT-ubuntu2004/python3. Warnings:
And 54 more Failing tests:
|
Build failed on mac12/noimt. Warnings:
And 53 more Failing tests:
|
Build failed on mac11/cxx14. Warnings:
And 54 more Failing tests:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
The
RooMomentMorphND
andRooMomentMorphFuncND
classes duplicated some copy-pasted code from stackoverflow. This is not factored out into a new private header file to avoid code duplication.Also, a semicolon is added after
TRACE_CREATE
andTRACE_DESTROY
in order to not confuse clang-format.