Changing exceptions to LogErrors where appropriate

[rappoccio]

Forward-port of #5803

[cmsbuild]

A new Pull Request was created by @rappoccio for CMSSW_7_3_X.

Changing exceptions to LogErrors where appropriate

It involves the following packages:

CondFormats/JetMETObjects

@apfeiffer1, @nclopezo, @monttj, @cmsbuild, @ggovi, @vadler can you please review it and eventually sign? Thanks.
@nhanvtran, @TaiSakuma, @mmarionncern, @schoef 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.
@nclopezo, @ktf you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[davidlange6]

Sal -do we need a 70x fix for this?

On Oct 13, 2014, at 9:47 PM, rappoccio notifications@github.com
wrote:

Forward-port of #5803

You can merge this Pull Request by running

git pull https://github.com/rappoccio/cmssw rappoccio_73x_jec_exception
Or view, comment on, or merge it at:

#5804

Commit Summary

• Changing exceptions to LogErrors where appropriate
File Changes

• M CondFormats/JetMETObjects/interface/JetCorrectorParameters.h (1)
• M CondFormats/JetMETObjects/interface/Utilities.h (4)
• M CondFormats/JetMETObjects/src/JetCorrectionUncertainty.cc (101)
• M CondFormats/JetMETObjects/src/SimpleJetCorrectionUncertainty.cc (22)
Patch Links:

• https://github.com/cms-sw/cmssw/pull/5804.patch
• https://github.com/cms-sw/cmssw/pull/5804.diff
—
Reply to this email directly or view it on GitHub.

[apfeiffer1]

should this logError not be inside an "else" or something ?

[cmsbuild]

+1
Tested at: d86f5ed
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-5804/52/summary.html

[cmsbuild]

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

[cmsbuild]

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

[vadler]

+1
Based on #5803 (comment) .

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_3_X IBs unless changes (tests are also fine). @nclopezo, @ktf can you please take care of it?