[clang-tools-extra] Fix #75687: add std::span to the default types. (PR #76116)

via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 20 18:02:00 PST 2023


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang-tidy

Author: None (Da-Viper)

<details>
<summary>Changes</summary>

Fixes #<!-- -->75687

---
Full diff: https://github.com/llvm/llvm-project/pull/76116.diff


2 Files Affected:

- (modified) clang-tools-extra/clang-tidy/readability/SimplifySubscriptExprCheck.cpp (+2-1) 
- (modified) clang-tools-extra/docs/clang-tidy/checks/readability/simplify-subscript-expr.rst (+1-1) 


``````````diff
diff --git a/clang-tools-extra/clang-tidy/readability/SimplifySubscriptExprCheck.cpp b/clang-tools-extra/clang-tidy/readability/SimplifySubscriptExprCheck.cpp
index d274abcbfabe8a..7d4698d27ed160 100644
--- a/clang-tools-extra/clang-tidy/readability/SimplifySubscriptExprCheck.cpp
+++ b/clang-tools-extra/clang-tidy/readability/SimplifySubscriptExprCheck.cpp
@@ -16,7 +16,8 @@ using namespace clang::ast_matchers;
 namespace clang::tidy::readability {
 
 static const char KDefaultTypes[] =
-    "::std::basic_string;::std::basic_string_view;::std::vector;::std::array";
+    "::std::basic_string;::std::basic_string_view;::std::vector;::std::array;::"
+    "std::span";
 
 SimplifySubscriptExprCheck::SimplifySubscriptExprCheck(
     StringRef Name, ClangTidyContext *Context)
diff --git a/clang-tools-extra/docs/clang-tidy/checks/readability/simplify-subscript-expr.rst b/clang-tools-extra/docs/clang-tidy/checks/readability/simplify-subscript-expr.rst
index f3f44bedcf74c5..4b7d7f2ddcf417 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/readability/simplify-subscript-expr.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/readability/simplify-subscript-expr.rst
@@ -20,4 +20,4 @@ Options
 .. option:: Types
 
    The list of type(s) that triggers this check. Default is
-   `::std::basic_string;::std::basic_string_view;::std::vector;::std::array`
+   `::std::basic_string;::std::basic_string_view;::std::vector;::std::array;::std::span`

``````````

</details>


https://github.com/llvm/llvm-project/pull/76116


More information about the cfe-commits mailing list