cleanup of uninitialized reads reported by valgrind (backport of #11173 )

[slava77]

fixing up what was supposed to be in #11176

A fraction of uninitialized value reads reported by valgrind in wflow 4.53 step3 is cleaned up.
The following summary is based on tests in 76X with #11173 (there is no ZDC monitor change in 74X; otherwise it's the same change as in #11173 )

• PileupJetIdAlgo is buggy and will need to be fixed by JME developers (I will start a separate thread for this). I have just gotten rid of uninitialized reads.
  • confirmed with valgrind as fixed
• B2GDQM was missing initialization of semiE_LepJetPtCut (the default uninitialized value was between zero and any other value; it now becomes 30 GeV as desired in the config). Corresponding plots will change.
  • confirmed with valgrind as fixed
• EcalClusterLazyToolsBase had reads beyond vector size (should be harmless from the logic)
  • confirmed with valgrind as fixed
• patPFMETCorrections changes appear to be cosmetic
  • the invalid read in SmearedJetProducerT calling StringCutObjectSelector still remains
• changes in cscdqm::StripClusterFinder are towards cleaning of uninitialized values in StripClusterFitData::height_
  • cleanup is not confirmed by valgrind

Tested in CMSSW_7_4_X_2015-09-07-1100 /test area sign584/:
- short matrix test comparisons show changes in affected plots in DQM
- reco products monitored with fwlite script are not affected

[slava77]

@cmsbuild please test

[cmsbuild]

The tests are being triggered in jenkins.

[cmsbuild]

A new Pull Request was created by @slava77 (Slava Krutelyov) for CMSSW_7_4_X.

cleanup of uninitialized reads reported by valgrind (backport of #11173 )

It involves the following packages:

DQM/CSCMonitorModule
DQM/Physics
DataFormats/JetReco
RecoEcal/EgammaCoreTools

@cmsbuild, @cvuosalo, @danduggan, @deguio, @slava77 can you please review it and eventually sign? Thanks.
@rappoccio, @Sam-Harper, @barvic, @ahinzmann, @TaiSakuma, @lgray, @jdolen, @nhanvtran, @schoef, @mariadalfonso, @argiro 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.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[cmsbuild]

+1
Tested at: d4f9a57
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-11177/7947/summary.html

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-11177/7947/summary.html

[slava77]

+1

for #11177 d4f9a57

• jenkins tests pass and comparisons with the baseline are in agreement with local tests (diffs are only in a while-since randomly changing DQM plots)

[deguio]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs once checked with relvals in the development release cycle of CMSSW (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

[davidlange6]

+1