aaron.ballman added inline comments.

================
Comment at: clang-tidy/misc/UserDefinedCopyWithoutAssignmentCheck.cpp:22
@@ +21,3 @@
+void UserDefinedCopyWithoutAssignmentCheck::registerMatchers(
+    MatchFinder *Finder) {
+  Finder->addMatcher(
----------------
Before registering the matchers, can you early return if not in CPlusPlus mode? 
(No need to register these matchers for C code.)

================
Comment at: clang-tidy/misc/UserDefinedCopyWithoutAssignmentCheck.cpp:63
@@ +62,3 @@
+void UserDefinedCopyWithoutAssignmentCheck::check(
+    const MatchFinder::MatchResult &Result) {
+  if (const auto *MatchedDecl =
----------------
This seems to have a lot of duplicate code that could be consolidated.

================
Comment at: clang-tidy/misc/UserDefinedCopyWithoutAssignmentCheck.h:21
@@ +20,3 @@
+///
+/// MSVC 2015 will generate an assignment operator even if the user defines a
+/// copy-constructor. This check finds classes with user-defined
----------------
Should update these comments to reflect the check's behavior better (and remove 
mention of MSVC since this isn't specific to that compiler).

================
Comment at: clang-tidy/misc/UserDefinedCopyWithoutAssignmentCheck.h:28
@@ +27,3 @@
+/// 
http://clang.llvm.org/extra/clang-tidy/checks/misc-user-defined-copy-without-assignment.html
+class UserDefinedCopyWithoutAssignmentCheck : public ClangTidyCheck { public:
+  UserDefinedCopyWithoutAssignmentCheck(StringRef Name, ClangTidyContext
----------------
This no longer is about just copy, so you may want to rename the class (and 
files and check) to something different.

================
Comment at: 
docs/clang-tidy/checks/misc-user-defined-copy-without-assignment.rst:8
@@ +7,3 @@
+constructor.  This behaviour is deprecated by the standard (C++ 14 draft
+standard 12.8.18)
+
----------------
Best to use the tags ([class.copy] paragraph 18) when referring to the 
standard, since the numbers can change.


http://reviews.llvm.org/D16376



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to