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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits