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
[RFC] Fix logical not in HLTHiggsPlotter::analyze #12880
Conversation
logical not is applied to unsigned int, which is a mistake: HLTriggerOffline/Higgs/src/HLTHiggsPlotter.cc:217:27: warning: logical not is only applied to the left hand side of comparison [-Wlogical-not-parentheses] HLTriggerOffline/Higgs/src/HLTHiggsPlotter.cc:217:27: warning: comparison of constant '6' with boolean expression is always false [-Wbool-compare] HLTriggerOffline/Higgs/src/HLTHiggsPlotter.cc:224:23: warning: logical not is only applied to the left hand side of comparison [-Wlogical-not-parentheses] HLTriggerOffline/Higgs/src/HLTHiggsPlotter.cc:224:23: warning: comparison of constant '6' with boolean expression is always false [-Wbool-compare] The patch attempts to resolve it. Signed-off-by: David Abdurachmanov <David.Abdurachmanov@cern.ch>
A new Pull Request was created by @davidlt for CMSSW_8_0_X. It involves the following packages: HLTriggerOffline/Higgs @cmsbuild, @vanbesien, @deguio, @davidlange6 can you please review it and eventually sign? Thanks. Following commands in first line of a comment are recognized
|
ping^1 |
I would ask @jasperlauwers to review the logic before approving. |
Please continue with the PR. |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_8_0_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @Degano, @smuzaffar |
+1 |
[RFC] Fix logical not in HLTHiggsPlotter::analyze
logical not is applied to unsigned int, which is a mistake:
The patch attempts to resolve it.
I am not sure if logical not should be applied only to
objType == EVTColContainer::PFJET
or a full condition in if-statement. My current assumption is that author intended to apply this forobjType == EVTColContainer::PFJET
only. This is why it's marked as RFC.Signed-off-by: David Abdurachmanov David.Abdurachmanov@cern.ch