en Copyright 2015 Java http://172.16.0.5:8080/history/llvm-project-15.0.7/clang-tools-extra/clang-tidy/bugprone/StringviewNullptrCheck.cpp#9edeceae [libTooling] Generalize string explanation as templated metadataChange RewriteRule from holding an `Explanation` to being able to generatearbitrary metadata. Where TransformerClangTidyCheck was interested in a stringdescription for the diagnostic, other tools may be interested in richer metadataat a higher level of abstraction than at the edit level (which is currentlyavailable as ASTEdit::Metadata).Reviewed By: ymandelDifferential Revision: https://reviews.llvm.org/D120360 List of files: /llvm-project-15.0.7/clang-tools-extra/clang-tidy/bugprone/StringviewNullptrCheck.cpp Mon, 21 Mar 2022 20:38:40 +0000 Eric Li <li.zhe.hua@gmail.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang-tools-extra/clang-tidy/bugprone/StringviewNullptrCheck.cpp#8351726e Revert "[libTooling] Generalize string explanation as templated metadata"This reverts commit 18440547d3520b78c9ab929685309419fc1fbe95. Causing failuresin some build modes.e.g. https://lab.llvm.org/buildbot/#/builders/217/builds/1886 List of files: /llvm-project-15.0.7/clang-tools-extra/clang-tidy/bugprone/StringviewNullptrCheck.cpp Mon, 21 Mar 2022 19:03:20 +0000 Yitzhak Mandelbaum <yitzhakm@google.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang-tools-extra/clang-tidy/bugprone/StringviewNullptrCheck.cpp#18440547 [libTooling] Generalize string explanation as templated metadataChange RewriteRule from holding an `Explanation` to being able to generatearbitrary metadata. Where TransformerClangTidyCheck was interested in a stringdescription for the diagnostic, other tools may be interested in richer metadataat a higher level of abstraction than at the edit level (which is currentlyavailable as ASTEdit::Metadata).Reviewed By: ymandelDifferential Revision: https://reviews.llvm.org/D120360 List of files: /llvm-project-15.0.7/clang-tools-extra/clang-tidy/bugprone/StringviewNullptrCheck.cpp Mon, 21 Mar 2022 18:19:17 +0000 Eric Li <li.zhe.hua@gmail.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang-tools-extra/clang-tidy/bugprone/StringviewNullptrCheck.cpp#a5684114 [clang-tidy] Update bugprone-stringview-nullptr to consistently prefer the empty string when passing arguments to constructors/functionsPreviously, function(nullptr) would have been fixed with function({}). This unfortunately can change overload resolution and even become ambiguous. T(nullptr) was already being fixed with T(""), so this change just brings function calls in line with that.Differential Revision: https://reviews.llvm.org/D117840 List of files: /llvm-project-15.0.7/clang-tools-extra/clang-tidy/bugprone/StringviewNullptrCheck.cpp Thu, 20 Jan 2022 23:05:07 +0000 CJ Johnson <johnsoncj@google.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang-tools-extra/clang-tidy/bugprone/StringviewNullptrCheck.cpp#7e29da87 Add support for return values in bugprone-stringview-nullptrbugprone-stringview-nullptr was not initially written with tests for return statements. After landing the check, the thought crossed my mind to add such tests. After writing them, I realized they needed additional handling in the matchers.Differential Revision: https://reviews.llvm.org/D115121 List of files: /llvm-project-15.0.7/clang-tools-extra/clang-tidy/bugprone/StringviewNullptrCheck.cpp Sun, 05 Dec 2021 19:11:02 +0000 CJ Johnson <johnsoncj@google.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang-tools-extra/clang-tidy/bugprone/StringviewNullptrCheck.cpp#6a9487df Add new clang-tidy check for string_view(nullptr)Checks for various ways that the `const CharT*` constructor of `std::basic_string_view` can be passed a null argument and replaces them with the default constructor in most cases. For the comparison operators, braced initializer list does not compile so instead a call to `.empty()` or the empty string literal are used, where appropriate.This prevents code from invoking behavior which is unconditionally undefined. The single-argument `const CharT*` constructor does not check for the null case before dereferencing its input. The standard is slated to add an explicitly-deleted overload to catch some of these cases: wg21.link/p2166https://reviews.llvm.org/D114823 is a companion change to prevent duplicate warnings from the `bugprone-string-constructor` check.Reviewed By: ymandelDifferential Revision: https://reviews.llvm.org/D113148 List of files: /llvm-project-15.0.7/clang-tools-extra/clang-tidy/bugprone/StringviewNullptrCheck.cpp Thu, 02 Dec 2021 13:17:12 +0000 CJ Johnson <johnsoncj@google.com>