Skip to content
Snippets Groups Projects

bugfix in the StsSetup

Merged Sergey Gorbunov requested to merge se.gorbunov/cbmroot:warnFixStsSetup into master

This MR fixes the following compiler warning:

/home/cbmdock/cbmroot/core/detectors/sts/CbmStsSetup.cxx: In member function 'void CbmStsSetup::RecomputePhysicalAssmbBbox(TGeoManager*)':
/home/cbmdock/cbmroot/core/detectors/sts/CbmStsSetup.cxx:382:59: warning: unused parameter 'geo' [-Wunused-parameter]
 void CbmStsSetup::RecomputePhysicalAssmbBbox(TGeoManager* geo)
                                              ~~~~~~~~~~~~~^~~

The problem is that RecomputePhysicalAssmbBbox() method uses gGeoManager instead of the input variable "geo". It doesn't matter much because, in the calling code, the "geo" variable is always initialized with gGeoManager. But for consistency, one should either remove the "geo" input variable and always use gGeoManager, or use the variable. I don't know which way is better. I rewrote the code so that it uses the variable.

Edited by Sergey Gorbunov

Merge request reports

Loading
Loading

Activity

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