[libc-commits] [PATCH] D112720: [clang-tidy] Use globs in HeaderFilter

Carlos Galvez via Phabricator via libc-commits libc-commits at lists.llvm.org
Thu Oct 28 08:07:52 PDT 2021


carlosgalvezp added a comment.

In D112720#3093636 <https://reviews.llvm.org/D112720#3093636>, @lebedev.ri wrote:

> Not in favor of general directions like this.
> Does it not break the existing uses (existing `.clang-tidy`'s e.g.)

Yep, it's a breaking change. The purpose of this patch is mostly to show how it would look like and see if that would be the desired result. There's 2 aspects of it:

- Rename of `HeaderFilterRegex` to `HeaderFilter`, to be consistent with `--header-filter`. If this is too much breakage, I'm OK with reverting.
- Replace regex with globs. Won't break the use of `.*`. It will break for the use case of having a parent `.clang-tidy` with `.*` and a child `.clang-tidy` with `my_folder/*\.h`. I personally always found the current interface confusing as a user. What regex should I write? Should I consider full paths, relative paths, does it matter?

I suppose I can keep the exact same behavior as master, but that will add additional tweaks and complexity to the `GlobList` class.


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

https://reviews.llvm.org/D112720



More information about the libc-commits mailing list