njames93 added a comment.
In D75621#1905879 <https://reviews.llvm.org/D75621#1905879>, @jroelofs wrote:
> Preserve backwards compatibility of ',' as a delimiter (for now).
>
> > The llvm::StringRef::split function can take multiple split characters,
>
> AFAIU, that's for multi-character delimiters, not multiple delimiters.
Yeah just checked it out my bad.
================
Comment at: clang-tools-extra/clang-tidy/utils/FileExtensionsUtils.cpp:41
+ for (const char Delimiter : Delimiters) {
+ if (StringRef::npos != AllFileExtensions.find(Delimiter)) {
+ if (Delimiter == ',') {
----------------
`if (AllFileExtensions.contains(Delimeter)) {`
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75621/new/
https://reviews.llvm.org/D75621
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
