Replacing runtime cms::Exception calls with edm::LogError calls in the jet flavour code

[ferencek]

Following discussion in the RECO/AT meeting on May 8, 2014 (https://indico.cern.ch/event/317592/#2014-05-08), this PR replaces runtime cms::Exception calls with edm::LogError calls in the jet flavour code

[cmsbuild]

A new Pull Request was created by @ferencek (Dinko Ferencek) for CMSSW_7_1_X.

Replacing runtime cms::Exception calls with edm::LogError calls in the jet flavour code

It involves the following packages:

PhysicsTools/JetMCAlgos

@nclopezo, @vadler, @cmsbuild, @Degano, @monttj can you please review it and eventually sign? Thanks.
@pvmulder 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.

[cmsbuild]

+1
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3785/1428/summary.html

[cmsbuild]

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

[vadler]

S.

• New jet flavour definition: Backport from 7_1_X to 7_0_X #3256 (diff)
• New jet flavour definition: Backport from 7_1_X to 7_0_X #3256 (diff)
• New jet flavour definition: Backport from 7_1_X to 7_0_X #3256 (diff)
  This is of course not visible here, since we have separate PRs in 71X

[cmsbuild]

Pull request #3785 was updated. @nclopezo, @vadler, @cmsbuild, @Degano, @monttj can you please check and sign again.

[vadler]

I can see #3785 (comment) , but I don't see a corresponding commit. Should that be the fix for #3785 (comment) ?

[cmsbuild]

+1
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3785/1452/summary.html

[cmsbuild]

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

[ferencek]

I think #3785 (comment) was a delayed automatic response on the last commit added to this branch which cleans up the code a bit.

[vadler]

+1

[cmsbuild]

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