<div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 17, 2012 at 5:18 PM, Daniel Jasper <span dir="ltr"><<a href="mailto:djasper@google.com" target="_blank">djasper@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div>The patch looks good to me aside from the issues below.</div><div><br></div><div>  +//  FIXME: The strategy forward is to provide a plugin system that can load</div><div>  +//  custom compilation databases and make enabling that a build option.</div>

</div><div><br></div><div>I think this is either a leftover or needs to be adapted to the init stuff.</div></blockquote><div>Yep, seems to be irrelevant.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>  +#ifndef LLVM_TOOLS_CLANG_LIB_TOOLING_CUSTOMTOOLINIT_H_</div><div><br></div><div>This should be LLVM_CLANG_TOOLING_CUSTOM_TOOL_INIT_H</div>
<div>We leave out TOOLS and LIB, don't put "_" at the end and separate CamelCase by "_".</div></blockquote><div>Done, but I didn't even think that there's a rule for this, and didn't find it now. Would you be so kind to point me to a relevant document?  </div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>  +void customToolInit(int argc, const char **argv);</div></div><div><br></div>
<div>I think we should put this into the clang::tooling:: namespace, just in case. Could you then also put the findCompilationDatabaseForDirectory() into that namespace?</div></blockquote><div>Makes sense. Done. </div><div>
<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>  +#ifndef LLVM_TOOLS_CLANG_LIB_TOOLING_CUSTOMCOMPILATIONDATABASE_H_</div>
<div><br></div><div>Thanks for adding this (/me forgot), but it should be:</div><div>LLVM_CLANG_TOOLING_CUSTOM_COMPILATION_DATABASE_H</div></blockquote><div>Done.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div>  +#ifdef USE_CUSTOM_TOOL_INIT</div><div>  +#include "clang/Tooling/CustomToolInit.h"</div>
<div>  +#endif</div></div><div>Have you actually tried compiling with that option? I think it can't find the header file like that as there is no such directory. Simply do '#include "CustomToolInit.h"'.</div>
</blockquote><div>Yep, my bad. Changed.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br>
<div class="gmail_quote">On Tue, Jul 17, 2012 at 4:47 PM, Alexander Kornienko <span dir="ltr"><<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This patch adds a custom initialize hook for clang tools. This is useful for plugging in custom initialization in private code bases and IDEs. Currently it can be used by compiling with a -DUSE_CUSTOM_TOOL_INIT and linking in an implementation for the customToolInit function.<span><font color="#888888"><br>


<div><div><br></div>-- <br>
<div>Regards,</div><div>Alexander</div></div>
</font></span></blockquote></div><br></div>
</div></div></blockquote></div><br>-- <br>
</div><div class="gmail_extra">Regards,</div><div class="gmail_extra">Alexander</div>