Skip to content
Snippets Groups Projects

CheckPV fix so Print actually works

Merged Patrick Koppenburg requested to merge pkoppenb-CheckPV into master
2 unresolved threads

The line

   if ( msgLevel( MSG::VERBOSE ) || m_print ) verbose() << "There are " << n << " primary vertices." << endmsg;

is nonsensical as m_print will only take effect in verbose mode. The fix is to print always() for m_print as that's the intention.

Edited by Patrick Koppenburg

Merge request reports

Pipeline #2160482 passed

Pipeline passed for 4bebf135 on pkoppenb-CheckPV

Approval is optional

Merged by Sebastien PonceSebastien Ponce 4 years ago (Dec 11, 2020 7:16am UTC)

Merge details

  • Changes merged into with 4e76e910.
  • Deleted the source branch.

Pipeline #2169203 passed

Pipeline passed for 4e76e910 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading