[PATCH] D34654: Allow passing a regex for headers to exclude from clang-tidy

[Mateusz via Phabricator via cfe-commits]

Ashimaru added a comment.

In D34654#3022728 , @alexfh wrote:

> Repeating my question from an earlier comment: would a header glob list 
> (similar to how the format of the `-checks=` flag) be enough for the use 
> cases folks have? On the one hand glob list doesn't support all the features 
> of regular expressions, but are they all useful for matching paths? Glob list 
> in clang-tidy currently supports set union (similar to regex `|`) and  subsequence> - `*` (regex `.*`). If needed, the support can be expanded with 
> `?`, character classes, character ranges and/or other features of POSIX globs 
> (https://man7.org/linux/man-pages/man7/glob.7.html). On the flipside, glob 
> list has a cleaner syntax (no need to quote characters common in paths - like 
> `.`), and allows to easily express exclusion of subsets. It should be a 
> convenient tool to represent a set of files / directories. In comparison to 
> the proposed header-filter + exclude-header-filter glob list makes it 
> possible to naturally express restrictions similar to "everything under a/ 
> (except for everything under a/b/ (except for everything under a/b/c/))" - 
> `a/,-a/b/,a/b/c/`.
>
> What do folks think?

Yes in my case this would be enough :)


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D34654/new/

https://reviews.llvm.org/D34654

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

[PATCH] D34654: Allow passing a regex for headers to exclude from clang-tidy

[Mateusz via Phabricator via cfe-commits]

Ashimaru added a comment.

In my company we are using a lot of standard checks and it becomes time 
consuming - this patch would be extremely useful to us and non of proposals 
montioned @alexfh seem to solve the issue.
We would like to select which directories containing headers would be checked 
and using filtering after checking still has problem of running all checks on 
file that will later discarded - and we do not like paying for something we do 
not use ;)


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D34654/new/

https://reviews.llvm.org/D34654

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