[PATCH] D26750: [clang-tidy] Add modernize-use-default-member-init check

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 17 05:59:05 PST 2016


aaron.ballman added inline comments.


================
Comment at: docs/clang-tidy/checks/modernize-use-default-member-init.rst:7
+This check converts a default constructor's member initializers into default
+member initializers.  Other member initializers that match the default
+member initializer are removed.  This can reduce repeated code or allow use of
----------------
malcolm.parsons wrote:
> Eugene.Zelenko wrote:
> > Please fix double spaces. Same below.
> If you like, but what harm are they causing?
Consistency; double-spacing is atypical in our documentation, probably because it's generally viewed online rather than in print.


Repository:
  rL LLVM

https://reviews.llvm.org/D26750





More information about the cfe-commits mailing list