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
Removed the need for const_cast in SiStripObjects #7378
Removed the need for const_cast in SiStripObjects #7378
Conversation
The static analyzer was complaining about the use of const_cast on member data. Even though these const_casts were in non-const member functions it aids future maintenance to remove their use.
A new Pull Request was created by @Dr15Jones (Chris Jones) for CMSSW_7_4_X. Removed the need for const_cast in SiStripObjects It involves the following packages: CalibFormats/SiStripObjects @diguida, @cerminar, @cmsbuild, @nclopezo, @rcastello, @mmusich can you please review it and eventually sign? Thanks. |
@cmsbuild please test |
The tests are being triggered in jenkins. |
-1 runTheMatrix-results/5.1_TTbar+TTbarFS+HARVESTFS/step1_TTbar+TTbarFS+HARVESTFS.log ----- Begin Fatal Exception 27-Jan-2015 12:18:58 CET----------------------- An exception of category 'ProductNotFound' occurred while [0] Processing run: 1 lumi: 1 event: 1 [1] Running path 'simulation_step' [2] Calling event method for module HBHEIsolatedNoiseReflagger/'hbhereco' Exception Message: Principal::getByToken: Found zero products matching all criteria Looking for type: std::vector Looking for module label: trackExtrapolator Looking for productInstanceName: Additional Info: [a] If you wish to continue processing events after a ProductNotFound exception, add "SkipEvent = cms.untracked.vstring('ProductNotFound')" to the "options" PSet in the configuration. ----- End Fatal Exception ------------------------------------------------- 135.4 step1 runTheMatrix-results/135.4_ZEE_13+ZEEFS_13+HARVESTUP15FS+MINIAODMCUP15FS/step1_ZEE_13+ZEEFS_13+HARVESTUP15FS+MINIAODMCUP15FS.log ----- Begin Fatal Exception 27-Jan-2015 12:20:07 CET----------------------- An exception of category 'ProductNotFound' occurred while [0] Processing run: 1 lumi: 1 event: 1 [1] Running path 'simulation_step' [2] Calling event method for module HBHEIsolatedNoiseReflagger/'hbhereco' Exception Message: Principal::getByToken: Found zero products matching all criteria Looking for type: std::vector Looking for module label: trackExtrapolator Looking for productInstanceName: Additional Info: [a] If you wish to continue processing events after a ProductNotFound exception, add "SkipEvent = cms.untracked.vstring('ProductNotFound')" to the "options" PSet in the configuration. ----- End Fatal Exception ------------------------------------------------- you can see the results of the tests here: |
@Dr15Jones again errors unrelated to the change... |
@cmsbuild please test |
The tests are being triggered in jenkins. |
@cerminar @diguida @mmusich @davidlange6 It's been a week |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs unless changes (tests are also fine). This pull request will be automatically merged. |
…ects Removed the need for const_cast in SiStripObjects
The static analyzer was complaining about the use of const_cast on
member data. Even though these const_casts were in non-const member
functions it aids future maintenance to remove their use.