ayartsev added a comment.

In http://reviews.llvm.org/D14629#288478, @jroelofs wrote:

> `CommonStuff.pm` (which could stand to have a more descriptive name, 
> `libscanbuild.pm` maybe?) and `scan-build.cfg` belong in `share`.


Got it, I'll rename `CommonStuff.pm` and put it to `share` in the next patch 
iteration.

In http://reviews.llvm.org/D14629#288485, @jroelofs wrote:

> Also, how is this different from `-analyzer-config`?


`-analyzer-config` is used to transfers a number of options to the analyzer, 
while configuration file is used to customize `scan-build`, `ccc-analyzer` and 
`c++-analyzer` scripts.

In http://reviews.llvm.org/D14629#288495, @jroelofs wrote:

> Actually, now that I think about it more, `scan-build.cfg` doesn't belong in 
> share at all. The location of an instance of it should be read in via a 
> command-line argument, and it shouldn't be installed.


I intend to add a command-line argument to customize the configuration file 
location, but there must be some default example configuration file to describe 
all available options and rules (`scan-build.cfg` from this patch).

Thanks for looking at this!


http://reviews.llvm.org/D14629



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to