<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Oct 25, 2013 at 11:45 AM, John Thompson <span dir="ltr"><<a href="mailto:john.thompson.jtsoftware@gmail.com" target="_blank">john.thompson.jtsoftware@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  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?<br></blockquote><div><br></div><div>The "old style" capitalized functions, and this is just a carry-over. I see now that these names are just matching the virtual function names, in which case I think that the way you have it is actually just fine.</div>
<div><br></div><div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
<br>
================<br>
Comment at: pp-trace/PPCallbacksTracker.cpp:176<br>
@@ +175,3 @@<br>
+                                      const clang::Module *Imported) {<br>
+  beginCallback("moduleImport");<br>
+  appendArgument("ImportLoc", ImportLoc);<br>
----------------<br>
</div><div class="im">Sean Silva wrote:<br>
> Any particular reason why the capitalization for this is different from the rest?<br>
</div>It's like this in the base class.  I'll fix it in a separate checkin.<br>
<br>
<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D2020" target="_blank">http://llvm-reviews.chandlerc.com/D2020</a><br>
</blockquote></div><br></div></div>