Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[TMVA] Use R__ASSERT instead of assert #13831

Merged
merged 1 commit into from
Oct 10, 2023
Merged

[TMVA] Use R__ASSERT instead of assert #13831

merged 1 commit into from
Oct 10, 2023

Conversation

dpiparo
Copy link
Member

@dpiparo dpiparo commented Oct 10, 2023

to fix warnings due to an unused variable in non-debug builds.

This Pull request:

Changes or fixes:

compilation warnings of TMVA

Checklist:

  • [v] tested changes locally
  • updated the docs (if necessary)

@dpiparo
[TMVA] Use R__ASSERT instead of assert
105957d 
to fix warnings due to an unused variable in non-debug builds.
@dpiparo dpiparo added this to the 6.32/00 milestone Oct 10, 2023
@dpiparo dpiparo self-assigned this Oct 10, 2023
@phsft-bot
Copy link
Collaborator

Starting build on ROOT-performance-centos8-multicore/soversion, ROOT-ubuntu2204/nortcxxmod, ROOT-ubuntu2004/python3, mac11/noimt, mac12arm/cxx20, windows10/default
How to customize builds

@dpiparo dpiparo merged commit 6b9eda4 into root-project:master Oct 10, 2023
8 of 13 checks passed
This was referenced Oct 10, 2023
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lmoneta lmoneta Awaiting requested review from lmoneta lmoneta is a code owner

@sitongan sitongan Awaiting requested review from sitongan

Assignees

@dpiparo dpiparo

Labels
None yet
Projects
None yet
Milestone
6.32/00
Development

Successfully merging this pull request may close these issues.
2 participants
@dpiparo @phsft-bot