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
Removed unnecessary const_cast in DataFormats/L1GlobalTrigger #11746
Removed unnecessary const_cast in DataFormats/L1GlobalTrigger #11746
Conversation
Removed const_cast which were then followed by reinterpret_cast but whose value was bound to a const variable. Adding const to the reinterpret_cast removed the need for const_cast. This was found by the static analyzer.
A new Pull Request was created by @Dr15Jones (Chris Jones) for CMSSW_7_6_X. Removed unnecessary const_cast in DataFormats/L1GlobalTrigger It involves the following packages: DataFormats/L1GlobalTrigger @cmsbuild, @mulhearn can you please review it and eventually sign? Thanks. |
Please test |
The tests are being triggered in jenkins. |
-1 Tested at: c520df4 ---> test TestConfigDP had ERRORS you can see the results of the tests here: The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
please test |
The tests are being triggered in jenkins. |
…GlobalTrigger Removed unnecessary const_cast in DataFormats/L1GlobalTrigger
Removed const_cast which were then followed by reinterpret_cast
but whose value was bound to a const variable. Adding const to
the reinterpret_cast removed the need for const_cast.
This was found by the static analyzer.