[PATCH] Add support for a few Objective-C matchers.
Manuel Klimek
klimek at google.com
Tue Mar 3 02:49:05 PST 2015
Apart from the nits LG. Please fix those before submitting.
================
Comment at: include/clang/ASTMatchers/ASTMatchers.h:2035-2039
@@ +2034,7 @@
+/// matches the [webView ...] message invocation.
+/// \code
+/// NSString *webViewJavaScript = ...
+/// UIWebView *webView = ...
+/// [webView stringByEvaluatingJavaScriptFromString:webViewJavascript];
+/// \endcode
+AST_MATCHER_P(ObjCMessageExpr, hasReceiverType, internal::Matcher<QualType>,
----------------
Please indent the code sample by 2 (make it consistent).
================
Comment at: include/clang/ASTMatchers/ASTMatchers.h:2105
@@ +2104,3 @@
+/// webView.frame = bodyFrame;
+/// // ^---- matches here
+/// \endcode
----------------
Can you make the comment-in-the-comment align with the start of the statement above?
http://reviews.llvm.org/D7710
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the cfe-commits
mailing list