Hey,

Is it OK if I commit the following:

This patch streamlines CheckerVisitor.def so that it follows the usual
'#ifndef foo' pattern:

#ifndef foo
#define foo(parm1)
#endif

foo(parm1) parm1 ...

#undef foo

-- 
Best regards,
Kovarththanan Rajaratnam
Index: include/clang/Analysis/PathSensitive/CheckerVisitor.def
===================================================================
--- include/clang/Analysis/PathSensitive/CheckerVisitor.def	(revision 89763)
+++ include/clang/Analysis/PathSensitive/CheckerVisitor.def	(working copy)
@@ -11,7 +11,14 @@
 //
 //===---------------------------------------------------------------------===//
 
-#ifdef PREVISIT
+#ifndef PREVISIT
+#define PREVISIT(NODE)
+#endif
+
+#ifndef POSTVISIT
+#define POSTVISIT(NODE)
+#endif
+
 PREVISIT(ArraySubscriptExpr)
 PREVISIT(BinaryOperator)
 PREVISIT(CallExpr)
@@ -19,12 +26,9 @@
 PREVISIT(DeclStmt)
 PREVISIT(ObjCMessageExpr)
 PREVISIT(ReturnStmt)
-#undef PREVISIT
-#endif
 
-#ifdef POSTVISIT
 POSTVISIT(CallExpr)
 POSTVISIT(BinaryOperator)
+
+#undef PREVISIT
 #undef POSTVISIT
-#endif
-
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to