CJ-Johnson added inline comments.
================
Comment at:
clang-tools-extra/test/clang-tidy/checkers/bugprone-stringview-nullptr.cpp:94-95
+
+// TODO: Handle cases where types, such as Class and Struct below, are
+// constructed with null arguments.
+class Class {
----------------
I attempted to address this TODO but it is quite the rabbit hole. If I just add
a basic fallback matcher, `applyFirst(...)` does not have the behavior I
desire. It was stomping on other, better fixes.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D115121/new/
https://reviews.llvm.org/D115121
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
