Avoid warning on clang #16438
Avoid warning on clang #16438
Conversation
Use () instead of {} when initializing atomic to avoid a warning
from clang because of how the symbol ATOMIC_FLAG_INIT is defined.
|
A new Pull Request was created by @Dr15Jones (Chris Jones) for CMSSW_8_1_X. It involves the following packages: FWCore/Concurrency @cmsbuild, @smuzaffar, @Dr15Jones, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here #13028 |
|
please test |
|
+1 |
cmsbuild
added
core-approved
fully-signed
and removed
core-pending
pending-signatures
labels
|
The tests are being triggered in jenkins. |
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_8_1_X IBs after it passes the integration tests. This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @smuzaffar |
|
This should substantially decrease the warnings from the clang build. |
|
-1 Tested at: 346cf6c The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: You can see the results of the tests here: I found follow errors while testing this PR Failed tests: UnitTests
I found errors in the following unit tests: ---> test runtestRecoLocalCaloHGCalRecProducers had ERRORS
There was error running material budget tests. The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
|
Comparison job queued. |
|
The test failures are unrelated to this change. |
Use () instead of {} when initializing atomic to avoid a warning
from clang because of how the symbol ATOMIC_FLAG_INIT is defined.