Skip to content

Commit

Permalink
[clang-tidy] Deprecate cert-dcl21-cpp
Browse files Browse the repository at this point in the history
It is no longer part of the CERT standard. Looking at the
CERT webpage, we can see it has been moved to the Void
section:
https://wiki.sei.cmu.edu/confluence/display/cplusplus/5+The+Void

Differential Revision: https://reviews.llvm.org/D147563
  • Loading branch information
carlosgalvezp committed Apr 5, 2023
1 parent 99fc6ec commit 712dfec
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 0 deletions.
3 changes: 3 additions & 0 deletions clang-tools-extra/docs/ReleaseNotes.rst
Original file line number Diff line number Diff line change
Expand Up @@ -183,6 +183,9 @@ Changes in existing checks
<clang-tidy/checks/bugprone/use-after-move>` check to also cover constructor
initializers.

- Deprecated :doc:`cert-dcl21-cpp
<clang-tidy/checks/cert/dcl21-cpp>` check.

- Deprecated check-local options `HeaderFileExtensions`
in :doc:`google-build-namespaces
<clang-tidy/checks/google/build-namespaces>` check.
Expand Down
4 changes: 4 additions & 0 deletions clang-tools-extra/docs/clang-tidy/checks/cert/dcl21-cpp.rst
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,10 @@
cert-dcl21-cpp
==============

.. note::
This check is deprecated since it's no longer part of the CERT standard.
It will be removed in :program:`clang-tidy` version 19.

This check flags postfix ``operator++`` and ``operator--`` declarations
if the return type is not a const object. This also warns if the return type
is a reference type.
Expand Down

0 comments on commit 712dfec

Please sign in to comment.