[PATCH] D133244: [clang-tidy] Readability-container-data-pointer adds new option to ignore Containers
Piotr Zegar via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jun 7 23:00:09 PDT 2023
PiotrZSL accepted this revision.
PiotrZSL added a comment.
This revision is now accepted and ready to land.
LGTM (+-)
- Before committing rebase this on top of main branch (looks like this base on previous release branch.
- Keep in mind 80 characters limit for documentation files
================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:179-181
+- Improved `readability-container-data-pointer <clang-tidy/checks/readability/container-data-pointer>` check.
+
+ The check now skips containers that are defined in the option `IgnoredContainers`.
----------------
Rebase this change on top of main branch, and follow other checks improvements format.
================
Comment at: clang-tools-extra/docs/clang-tidy/checks/readability/container-data-pointer.rst:20
+
+ Semicolon-separated list of containers regexp for which container-data-pointer check won't be enforced. Default is `empty`.
----------------
80 characters per line, split this, do not duplicate check name.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D133244/new/
https://reviews.llvm.org/D133244
More information about the cfe-commits
mailing list