Skip to content

Fix several calng-tidy warnings in TMVA#7592

Merged
linev merged 3 commits intoroot-project:masterfrom
linev:tmva_tidy
Mar 19, 2021
Merged

Fix several calng-tidy warnings in TMVA#7592
linev merged 3 commits intoroot-project:masterfrom
linev:tmva_tidy

Conversation

@linev
Copy link
Copy Markdown
Member

@linev linev commented Mar 18, 2021

Fixes #7526

@phsft-bot
Copy link
Copy Markdown

Starting build on ROOT-debian10-i386/cxx14, ROOT-performance-centos8-multicore/default, ROOT-fedora30/cxx14, ROOT-fedora31/noimt, ROOT-ubuntu16/nortcxxmod, mac1014/python3, mac11.0/cxx17, windows10/cxx14
How to customize builds

@phsft-bot
Copy link
Copy Markdown

Build failed on windows10/cxx14.
Running on null:C:\build\workspace\root-pullrequests-build
See console output.

Copy link
Copy Markdown
Member

@lmoneta lmoneta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!
Thank you for this PR

@linev linev merged commit 8c16e62 into root-project:master Mar 19, 2021
@linev linev deleted the tmva_tidy branch March 19, 2021 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[tmva] Clang-tidy Clazy warnings

3 participants