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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
