[PATCH] D126891: [clang-tidy] The check should ignore final classes

Whisperity via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jun 19 23:36:14 PDT 2022


whisperity accepted this revision.
whisperity added a comment.
This revision is now accepted and ready to land.

Thank you!



================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:170
+  <clang-tidy/checks/cppcoreguidelines-virtual-class-destructor>` involving
+  ``final`` classes. The check will not diagnose ``final`` marked classes, since
+  those cannot be used as base classes, consequently they can not violate the
----------------
(Nit: classes marked final)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D126891/new/

https://reviews.llvm.org/D126891



More information about the cfe-commits mailing list