[clang-tools-extra] [docs][clang-tidy] Correct StrictMode example in modernize-use-std-print (PR #108805)
via cfe-commits
cfe-commits at lists.llvm.org
Mon Sep 16 02:13:41 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang-tidy
Author: Mainak Sil (MainakSil)
<details>
<summary>Changes</summary>
Updated the example in the `StrictMode` section of the clang-tidy check `modernize-use-std-print`.
The previous example incorrectly swapped the cast of signed and unsigned integers. Specifically:
- The signed integer `i` was being cast to `unsigned int`, and
- The unsigned integer `u` was being cast to `int`.
This correction ensures that the behavior of `std::print` with `StrictMode` enabled matches that of `printf`, by reversing the casts to maintain the correct signedness.
Issue Refference
It solves #<!-- -->101397
---
Full diff: https://github.com/llvm/llvm-project/pull/108805.diff
1 Files Affected:
- (modified) clang-tools-extra/docs/clang-tidy/checks/modernize/use-std-print.rst (+1-1)
``````````diff
diff --git a/clang-tools-extra/docs/clang-tidy/checks/modernize/use-std-print.rst b/clang-tools-extra/docs/clang-tidy/checks/modernize/use-std-print.rst
index 59bb722e2c24fc..a825cd7432a57d 100644
--- a/clang-tools-extra/docs/clang-tidy/checks/modernize/use-std-print.rst
+++ b/clang-tools-extra/docs/clang-tidy/checks/modernize/use-std-print.rst
@@ -109,7 +109,7 @@ Options
.. code-block:: c++
- std::print("{} {}\n", static_cast<unsigned int>(i), static_cast<int>(u));
+ std::print("{} {}\n", static_cast<int>(u), static_cast<unsigned int>(i));
to ensure that the output will continue to be the unsigned representation
of `-42` and the signed representation of `0xffffffff` (often
``````````
</details>
https://github.com/llvm/llvm-project/pull/108805
More information about the cfe-commits
mailing list