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
Made staic debug controls std::atomics #4421
Made staic debug controls std::atomics #4421
Conversation
To avoid race conditions when changing the debug conditions, switched all the variables to be std::atomics following similar code use in other classes of this package. These statics were found by the static analyzer.
A new Pull Request was created by @Dr15Jones (Chris Jones) for CMSSW_7_2_X. Made staic debug controls std::atomics It involves the following packages: EventFilter/CSCRawToDigi @nclopezo, @cmsbuild, @Degano, @StoyanStoynev, @slava77 can you please review it and eventually sign? Thanks. |
@barvic |
LOCAL_UNPACK is for out-of-CMSSW framework compilation. If proposed changes pass current general CMSSW tests, please just proceed with PR signing. |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes (tests are also fine). |
…RawToDigi Made staic debug controls std::atomics
To avoid race conditions when changing the debug conditions, switched
all the variables to be std::atomics following similar code use in
other classes of this package.
These statics were found by the static analyzer.