make statics class members

[gartung]

No description provided.

[cmsbuild]

A new Pull Request was created by @gartung (Patrick Gartung) for CMSSW_7_0_X.

make statics class members

It involves the following packages:

RecoMET/METAlgorithms

@nclopezo, @cmsbuild, @thspeer, @slava77 can you please review it and eventually sign? Thanks.
@TaiSakuma 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.
@ktf you are the release manager for this.

[Dr15Jones]

If the CSCHaloAlgo is made every every (which I haven't checked) then you'd get the same message every event, which that static would have avoided. If the CSCHaloAlgo sticks around a long time, it's not a big deal.

[cmsbuild]

Pull request #1430 was updated. @nclopezo, @cmsbuild, @thspeer, @slava77 can you please check and sign again.

[Dr15Jones]

On each of the `if`` blocks you will want to change them to

bool expected=false;
if(MuonTimeFail.compare_exchange_strong(expected,true,std::memory_order_acq_rel) ) {

and then get rid of the MuonTimeFail = true.

[cmsbuild]

Pull request #1430 was updated. @nclopezo, @cmsbuild, @thspeer, @slava77 can you please check and sign again.

[Dr15Jones]

Looks good to me.

[cmsbuild]

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

[thspeer]

+1
Tested 063c9cb in CMSSW_7_0_X_2013-11-14-0200-1430
No difference in reco as expected, based on RelMon and reco script

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next IBs unless changes or unless it breaks tests. @ktf can you please take care of it?