[PATCH] D114995: clang-tidy: improve the 'modernize-use-default-member-init'

Malcolm Parsons via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 9 09:51:43 PST 2021


malcolm.parsons added a comment.

In D114995#3180475 <https://reviews.llvm.org/D114995#3180475>, @aaron.ballman wrote:

> was there a reason we didn't cover that case originally or was it an oversight/left for future work?

It was left for future work - by only considering the initializer list of the default constructor, clang-tidy did not have to work out what to do when the constructors don't agree on what value the member init should have.
The next step towards the full solution is to handle classes with only one non-special constructor.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114995



More information about the cfe-commits mailing list