[clang-tools-extra] [clang-tidy] Add a release note about unchecked-optional-access smart pointer caching (PR #122290)
Jan Voung via cfe-commits
cfe-commits at lists.llvm.org
Thu Jan 9 07:06:21 PST 2025
https://github.com/jvoung created https://github.com/llvm/llvm-project/pull/122290
With caching added in https://github.com/llvm/llvm-project/pull/120249,
inform in notes that the `IgnoreSmartPointerDereference` option shouldn't
be needed anymore.
Other caching also added earlier:
https://github.com/llvm/llvm-project/pull/112605
>From 342ff1a05caa6943fe8a86415f30b433ac30106f Mon Sep 17 00:00:00 2001
From: Jan Voung <jvoung at gmail.com>
Date: Thu, 9 Jan 2025 14:10:30 +0000
Subject: [PATCH] [clang-tidy] Add a release note about
unchecked-optional-access smart pointer caching
With caching added in https://github.com/llvm/llvm-project/pull/120249,
inform in notes that the `IgnoreSmartPointerDereference` option shouldn't
be needed anymore.
Other caching also added earlier:
https://github.com/llvm/llvm-project/pull/112605
---
clang-tools-extra/docs/ReleaseNotes.rst | 3 +++
1 file changed, 3 insertions(+)
diff --git a/clang-tools-extra/docs/ReleaseNotes.rst b/clang-tools-extra/docs/ReleaseNotes.rst
index 94e15639c4a92e..f325b9ecbe1547 100644
--- a/clang-tools-extra/docs/ReleaseNotes.rst
+++ b/clang-tools-extra/docs/ReleaseNotes.rst
@@ -232,6 +232,9 @@ Changes in existing checks
<clang-tidy/checks/bugprone/unchecked-optional-access>` to support
`bsl::optional` and `bdlb::NullableValue` from
<https://github.com/bloomberg/bde>_.
+ Fixed false positives from smart pointer accessors repeated in checking
+ ``has_value`` and accessing ``value`` are now fixed by now caching.
+ So the option `IgnoreSmartPointerDereference` should no longer be needed.
- Improved :doc:`bugprone-unhandled-self-assignment
<clang-tidy/checks/bugprone/unhandled-self-assignment>` check by fixing smart
More information about the cfe-commits
mailing list