[PATCH] D24339: [clang-tidy] Add check 'readability-redundant-member-init'

[Eugene Zelenko via cfe-commits]

Eugene.Zelenko added a subscriber: Eugene.Zelenko.
Eugene.Zelenko added a comment.

I think will be good idea to add cases when member is initialized in declaration and constructor, with same and different values.


================
Comment at: docs/clang-tidy/checks/readability-redundant-member-init.rst:4
@@ +3,3 @@
+readability-redundant-member-init
+==================================
+
----------------
This line length should be same as header above.

================
Comment at: docs/clang-tidy/checks/readability-redundant-member-init.rst:13
@@ +12,3 @@
+  // Explicitly initializing the member s is unnecessary.                                                      
+  class Foo {
+    public:
----------------
Please run Clang-format over example. Will be good idea to add empty line before private.

================
Comment at: test/clang-tidy/readability-redundant-member-init.cpp:33
@@ +32,3 @@
+
+
+struct NF1 {
----------------
Please remove extra empty-line.


Repository:
  rL LLVM

https://reviews.llvm.org/D24339