Skip to content

Commit

Permalink
[clang-tidy][doc] Deprecate the AnalyzeTemporaryDtors option
Browse files Browse the repository at this point in the history
It's not used anywhere, and we should not keep it
for eternity. Document it as deprecated and announce
its removal 2 releases later, so people have time
to update their .clang-tidy files.

Differential Revision: https://reviews.llvm.org/D141583
  • Loading branch information
carlosgalvezp committed Jan 13, 2023
1 parent 6c09a4e commit 0f5eb31
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
2 changes: 1 addition & 1 deletion clang-tools-extra/clang-tidy/ClangTidyOptions.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ template <> struct MappingTraits<ClangTidyOptions> {
IO.mapOptional("Checks", Options.Checks);
IO.mapOptional("WarningsAsErrors", Options.WarningsAsErrors);
IO.mapOptional("HeaderFilterRegex", Options.HeaderFilterRegex);
IO.mapOptional("AnalyzeTemporaryDtors", Ignored); // legacy compatibility
IO.mapOptional("AnalyzeTemporaryDtors", Ignored); // deprecated
IO.mapOptional("FormatStyle", Options.FormatStyle);
IO.mapOptional("User", Options.User);
IO.mapOptional("CheckOptions", Options.CheckOptions);
Expand Down
4 changes: 4 additions & 0 deletions clang-tools-extra/docs/ReleaseNotes.rst
Original file line number Diff line number Diff line change
Expand Up @@ -102,6 +102,10 @@ Improvements to clang-tidy
- Fix a minor bug in `add_new_check.py` to only traverse subdirectories
when updating the list of checks in the documentation.

- Deprecate the global configuration file option `AnalyzeTemporaryDtors`,
which is no longer in use. The option will be fully removed in
:program:`clang-tidy` version 18.

New checks
^^^^^^^^^^

Expand Down

0 comments on commit 0f5eb31

Please sign in to comment.