[PATCH] D14629: [analyzer] Configuration file for scan-build.

[Антон Ярцев via cfe-commits]

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