New clang checker

[gartung]

No description provided.

[ktf]

Can you provide instructions (even just a comment to this pull request) on how to use it? Thanks.

[cmsbuild]

A new Pull Request was created by @gartung (Patrick Gartung) for CMSSW_7_0_X.

New clang checker

It involves the following packages:

Utilities/StaticAnalyzers

@smuzaffar, @Dr15Jones, @ktf, @nclopezo can you please review it and eventually sign? Thanks.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
@ktf you are the release manager for this.

[gartung]

On 10/29/2013 3:05 AM, Giulio Eulisse wrote:

Can you provide instructions (even just a comment to this pull
request) on how to use it? Thanks.

—
Reply to this email directly or view it on GitHub
#1208 (comment).

USER_LLVM_CHECKERS="-enable-checker cms.FunctionChecker" scram b checker

[gartung]

On 10/29/2013 3:05 AM, Giulio Eulisse wrote:

Can you provide instructions (even just a comment to this pull
request) on how to use it? Thanks.

—
Reply to this email directly or view it on GitHub
#1208 (comment).

If you want to run ClassChecker , which limits the checks to data
classes see the script
Utilities/StaticAnalyzers/scripts/run_class_checker.sh
which will copy the needed classes.txt to ${CMSSW_BASE}/tmp/classes.txt

[ktf]

@nclopezo can you try to see if you manage to get a list of issues from Patrick instructions?

[cmsbuild]

Pull request #1208 was updated. @smuzaffar, @Dr15Jones, @ktf, @nclopezo can you please check and sign again.

[cmsbuild]

+1
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/1078/summary.html

[nclopezo]

Hi,

I checked out some random packages in my local environment with this pull request

DataFormats/FWLite
FWCore/Version
RecoLocalCalo/CaloRecCandCreator
RecoLocalCalo/Castor
RecoLocalCalo/Configuration
RecoLocalCalo/EcalDeadChannelRecoveryAlgos
RecoLocalCalo/EcalRecAlgos
RecoLocalCalo/EcalRecProducers
RecoLocalCalo/HcalLaserReco
RecoLocalCalo/HcalRecAlgos
RecoLocalCalo/HcalRecProducers
Utilities/StaticAnalyzers

then I ran

USER_LLVM_CHECKERS="-enable-checker cms.FunctionChecker" scram b checker

and you can see the results here:

https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/pr1208/llvm-analysis/2013-10-30-1/

Is this ok? or should I run it with other packages as well?

[ktf]

https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/pr1208/llvm-analysis/2013-10-30-1/

Is this ok? or should I run it with other packages as well?

Can you update the jenkins rule to always run also this checker? I'll now merge
this.

[Dr15Jones]

@gartung Patrick, how is this change meant to be used?

[gartung]

Really ClassChecker should be run instead of FunctionChecker
The following two commands are need to run ClassChecker

cp -p ${CMSSW_BASE}/src/Utilities/StaticAnalyzers/scripts/classes.txt ${CMSSW_BASE}/tmp/classes.txt
USER_LLVM_CHECKERS="-enable-checker optional.ClassChecker" scram b checker