lebedev.ri added a comment.

Not sure how `clang/docs/LibASTMatchersReference.html` is supposed to be 
updated.



================
Comment at: include/clang/ASTMatchers/ASTMatchers.h:3931
+
+/// \brief Matches all kind of assignment operators.
+///
----------------
Maybe
```
-/// \brief Matches all kind of assignment operators.
+/// \brief Matches all kinds of assignment operators.
```


================
Comment at: include/clang/ASTMatchers/ASTMatchers.h:3938
+/// \endcode
+AST_MATCHER(BinaryOperator, isAssignmentOperator) {
+  return Node.isAssignmentOp();
----------------
Needs unittests, probably in 
`clang/unittests/ASTMatchers/ASTMatchersNarrowingTest.cpp` ?


https://reviews.llvm.org/D38921



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

Reply via email to