[Asterisk-code-review] Example script for scan-build (the llvm static analyzer) (asterisk[13])

Diederik de Groot asteriskteam at digium.com
Fri Apr 24 10:18:47 CDT 2015


Diederik de Groot has posted comments on this change.

Change subject: Example script for scan-build (the llvm static analyzer)
......................................................................


Patch Set 6:

Just Checking.

Are you guys sure you actually want this script in the contrib/scripts directory. 

I originally thought of it as a example, how this scan-build could be run on your Continues Integration Environment, and possible on developers PC's. You guys should decide where this should live. I would be just fine if contrib/scripts is the directory where you guys want this.

FYI: scan-build is the llvm static analyzer, which is even more 'annoying' than clang, but very good at find all kinds of stuff, that might need attention.

Sorry if i am raising this point a little late in the process (cherry-picking).

-- 
To view, visit https://gerrit.asterisk.org/158
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I70405fe54e4be7dbfbcb62e291690069b88617a8
Gerrit-PatchSet: 6
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Diederik de Groot <dkgroot at talon.nl>
Gerrit-Reviewer: Diederik de Groot <dkgroot at talon.nl>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-HasComments: No



More information about the asterisk-code-review mailing list