Skip to content

Conversation

@HerrCai0907
Copy link
Contributor

No description provided.

@llvmbot
Copy link
Member

llvmbot commented Dec 13, 2024

@llvm/pr-subscribers-clang-tidy

@llvm/pr-subscribers-clang-tools-extra

Author: Congcong Cai (HerrCai0907)

Changes

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

1 Files Affected:

  • (modified) clang-tools-extra/docs/clang-tidy/checks/misc/unused-parameters.rst (+7-7)
diff --git a/clang-tools-extra/docs/clang-tidy/checks/misc/unused-parameters.rst b/clang-tools-extra/docs/clang-tidy/checks/misc/unused-parameters.rst
index 87b75579d97a7c..9321f651fd7054 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/misc/unused-parameters.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/misc/unused-parameters.rst
@@ -35,13 +35,13 @@ Options
 
 .. option:: StrictMode
 
-   When `false` (default value), the check will ignore trivially unused parameters,
-   i.e. when the corresponding function has an empty body (and in case of
-   constructors - no constructor initializers). When the function body is empty,
-   an unused parameter is unlikely to be unnoticed by a human reader, and
-   there's basically no place for a bug to hide.
+  When `false` (default value), the check will ignore trivially unused parameters,
+  i.e. when the corresponding function has an empty body (and in case of
+  constructors - no constructor initializers). When the function body is empty,
+  an unused parameter is unlikely to be unnoticed by a human reader, and
+  there's basically no place for a bug to hide.
 
 .. option:: IgnoreVirtual
 
-   Determines whether virtual method parameters should be inspected.
-   Set to `true` to ignore them. Default is `false`.
+  Determines whether virtual method parameters should be inspected.
+  Set to `true` to ignore them. Default is `false`.

@HerrCai0907 HerrCai0907 merged commit 7da4b6b into llvm:main Dec 13, 2024
12 checks passed
@HerrCai0907 HerrCai0907 deleted the fix/doc2 branch December 13, 2024 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants