[PATCH] D112916: Confusable identifiers detection

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 22 04:22:13 PST 2022


aaron.ballman added inline comments.


================
Comment at: clang-tools-extra/clang-tidy/misc/ConfusableTable/build_confusable_table.cpp:22
+  SmallVector<StringRef> Lines;
+  Content.split(Lines, '\n', -1, false);
+
----------------
We're testing this functionality in our downstream and noticed that this causes an issue when the repo is cloned with CRLF  line endings. Can you make this a bit more resilient to the platform line endings?


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

https://reviews.llvm.org/D112916



More information about the cfe-commits mailing list