[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