This revision was automatically updated to reflect the committed changes.
Closed by commit rCTE361550: [clang-tidy]: Add cert-oop54-cpp alias for
bugprone-unhandled-self-assignment (authored by ztamas, committed by ).
Changed prior to commit:
ztamas updated this revision to Diff 200775.
ztamas added a comment.
Herald added a subscriber: mgorny.
Link burprone module to cert module
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62192/new/
https://reviews.llvm.org/D62192
Files:
ztamas updated this revision to Diff 200714.
ztamas added a comment.
Fixed docs.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62192/new/
https://reviews.llvm.org/D62192
Files:
Eugene.Zelenko added a comment.
Please mention alias (in aliases list) in Release Notes.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62192/new/
https://reviews.llvm.org/D62192
___
cfe-commits
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
LGTM aside from some nits, thank you for this!
Comment at:
clang-tools-extra/clang-tidy/bugprone/UnhandledSelfAssignmentCheck.cpp:89-91
+AdditionalMatcher
ztamas created this revision.
Herald added subscribers: cfe-commits, xazax.hun.
Herald added a project: clang.
Added WarnOnlyIfThisHasSuspiciousField option to allow
to catch any copy assignment operator independently from
the container class's fields.
Added the cert alias using this option.