[clang-tools-extra] [clang-tidy] Add a release note about unchecked-optional-access smart pointer caching (PR #122290)
Piotr Zegar via cfe-commits
cfe-commits at lists.llvm.org
Sat Jan 11 00:32:11 PST 2025
================
@@ -232,6 +232,10 @@ 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``, by caching the locations returned
+ by the accessors. The option `IgnoreSmartPointerDereference` should no
+ longer be needed.
----------------
PiotrZSL wrote:
"The option `IgnoreSmartPointerDereference` should no longer be needed."
If option is not needed, they maybe could be removed, if still got use outside of mentioned scenario, then maybe just say that it no longer needed to cover this scenario.
https://github.com/llvm/llvm-project/pull/122290
More information about the cfe-commits
mailing list