[PATCH] D97132: [clang-tidy] Harden PreferMemberInitializerCheck

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 22 06:06:14 PST 2021


njames93 updated this revision to Diff 325428.
njames93 added a comment.

Fix potential crash if macros are used, Now we just don't try to emit a fix.
Fix issue where FixIt for member initializers would be inserted at the start of the constructor decl due to implicit member initializers confusing it.
Streamlined method for finding where to create member initializers when there's no member initializer list.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97132

Files:
  clang-tools-extra/clang-tidy/cppcoreguidelines/PreferMemberInitializerCheck.cpp
  clang-tools-extra/test/clang-tidy/checkers/cppcoreguidelines-prefer-member-initializer.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D97132.325428.patch
Type: text/x-patch
Size: 13842 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20210222/2238e888/attachment-0001.bin>


More information about the cfe-commits mailing list