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.

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

Static analysis warnings fixes #11119

Merged
merged 2 commits into from Sep 4, 2015

Conversation

civanch
Copy link
Contributor

@civanch civanch commented Sep 4, 2015

It is the same PR as 11097

@davidlt
Copy link
Contributor

davidlt commented Sep 4, 2015

@civanch this also fixed G4 related segfaults in DEVEL IBs?

@davidlange6

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 4, 2015

A new Pull Request was created by @civanch (Vladimir Ivantchenko) for CMSSW_7_6_ROOT64_X.

Static analysis warnings fixes

It involves the following packages:

SimCalorimetry/EcalSelectiveReadoutAlgos
SimG4CMS/Calo
SimG4CMS/FP420
SimG4CMS/Forward
SimG4CMS/HcalTestBeam
SimG4Core/Application
SimG4Core/CustomPhysics
SimG4Core/PhysicsLists
SimMuon/RPCDigitizer
SimTracker/Common
SimTracker/SiStripDigitizer

@cmsbuild, @civanch, @mdhildreth can you please review it and eventually sign? Thanks.
@jlagram, @makortel, @GiacomoSguazzoni, @gbenelli, @rovere, @argiro, @cnuttens, @prolay, @jhgoh, @cerati, @VinInn, @threus, @dgulhan, @LBeck this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.

@civanch
Copy link
Contributor Author

civanch commented Sep 4, 2015

@davidlt , yes, it should fix few IB crashes. Not for the first time static analysis helps.

@civanch
Copy link
Contributor Author

civanch commented Sep 4, 2015

please test

@civanch
Copy link
Contributor Author

civanch commented Sep 4, 2015

+1

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 4, 2015

The tests are being triggered in jenkins.

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 4, 2015

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_6_ROOT64_X IBs once checked with relvals in the development release cycle of CMSSW or unless it breaks tests.

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 4, 2015

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_6_ROOT64_X IBs once checked with relvals in the development release cycle of CMSSW or unless it breaks tests.

davidlt added a commit that referenced this pull request Sep 4, 2015
@davidlt davidlt merged commit 1cc1cc4 into cms-sw:CMSSW_7_6_ROOT64_X Sep 4, 2015
@davidlt
Copy link
Contributor

davidlt commented Sep 4, 2015

Let's put it in and see if it resolves the issues.

@davidlt
Copy link
Contributor

davidlt commented Dec 8, 2015

@davidlange6 @civanch do we need this in CMSSW_8_0_X? Is this just static analyzer cleanups?

@civanch
Copy link
Contributor Author

civanch commented Dec 8, 2015

@davidlt, it is written that this PR is the same as #11097, which has been integrated in 7_6_X, likely these fixes are in 8_0_X already. However, there are still a lot of static analysis complains in 8_0_X for SIM and I plan to reduce them in coming weeks.

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.

None yet

3 participants