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
Minor thread-safety improvements to HLTrigger/HLTcore #4747
Minor thread-safety improvements to HLTrigger/HLTcore #4747
Conversation
-Converted a static to const to avoid complaints from static analyzer. -Removed use of HLTConfigCounter since wasn't thread safe and wasn't actually doing anything useful.
A new Pull Request was created by @Dr15Jones (Chris Jones) for CMSSW_7_2_X. Minor thread-safety improvements to HLTrigger/HLTcore It involves the following packages: HLTrigger/HLTcore @perrotta, @cmsbuild, @nclopezo, @Martin-Grunewald, @fwyzard, @Degano can you please review it and eventually sign? Thanks. |
I get an github error:
?? |
@ktf this pull has the github glitch. What is the way you use to get around it? |
Hi, When this happens I tell cms-bot to do git cms-merge-topic Dr15Jones:threadSafetyImprovementsForHLTriggerHLTcore using the branch name instead of the pr number. |
+1 |
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). |
…TriggerHLTcore HLTrigger/HLTcore -- Minor thread-safety improvements
-Converted a static to const to avoid complaints from static analyzer.
-Removed use of HLTConfigCounter since wasn't thread safe and wasn't
actually doing anything useful.