On second thought, aren't function members supposed start with a lower case?  
Are the other functions wrong, or is there a convention for virtual/callback 
functions?


================
Comment at: pp-trace/PPCallbacksTracker.cpp:176
@@ +175,3 @@
+                                      const clang::Module *Imported) {
+  beginCallback("moduleImport");
+  appendArgument("ImportLoc", ImportLoc);
----------------
Sean Silva wrote:
> Any particular reason why the capitalization for this is different from the 
> rest?
It's like this in the base class.  I'll fix it in a separate checkin.



http://llvm-reviews.chandlerc.com/D2020
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to