<div dir="ltr">On Wed, Nov 6, 2013 at 7:39 AM, Alp Toker <span dir="ltr"><<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im"><br>
On 06/11/2013 14:55, Anthony Scopatz wrote:<br>
> Hello All,<br>
><br>
> Could we please hear some response on this?  Thanks!<br>
<br>
</div>Hello Anthony, Geoffrey,<br>
<br>
Thanks for the contribution.<br>
<br>
You'll need to rebase this work against current SVN, add at least a<br>
couple of standard lit tests and probably split it up into two or three<br>
bite-size chunks for review.<br>
<br>
Once that's done, it can also help to CC in others who've recently<br>
worked on the C API in order to get buy-in.<br></blockquote><div><br></div><div>Thanks Alp, </div><div><br></div><div>Do you have any suggestions on who to CC or should we just look at the logs?</div><div><br></div><div>

Thanks!</div><div><br></div><div>Be Well</div><div>Anthony</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
As for the Geoffrey's build problem, it's best to ask that as a separate<br>
question, including details about the platform and configuration if<br>
you're still struggling.<br>
<br>
Good luck with the patch :-)<br>
<br>
Alp.<br>
<div class="im"><br>
<br>
<br>
><br>
> Be Well<br>
> Anthony<br>
><br>
><br>
> On Sun, Nov 3, 2013 at 5:46 PM, Geoffrey Irving <<a href="mailto:irving@naml.us">irving@naml.us</a><br>
</div><div class="im">> <mailto:<a href="mailto:irving@naml.us">irving@naml.us</a>>> wrote:<br>
><br>
>     Here are four patches improving libclang in various ways for use in<br>
>     xdress (<a href="http://xdress.org" target="_blank">http://xdress.org</a>).  In order, they add<br>
><br>
>     1. Python support for clang_formatDiagnostic.<br>
>     2. Access to public / private / protected specifiers.<br>
>     3. Template argument access (as new cursor types).<br>
>     4. A clang_Cursor_getDefaultArgument function.<br>
><br>
>     They've all been tested inside xdress, but the patches themselves do<br>
>     not include tests.  Since questions about this stuff to cfe-dev were<br>
>     ignored, I want to make sure these patches will be looked at before I<br>
>     do potentially unnecessary work.<br>
><br>
>     Caveat: llvm trunk does not build for me due to these errors:<br>
><br>
>         llvm-build: error: invalid native target: 'x86_64' (not in<br>
>     project)<br>
><br>
>     Therefore, these patches were written, built, and tested again<br>
>     release_33 plus several cherry-picked libclang commits.  It's possible<br>
>     they don't compile against the trunk.<br>
><br>
>     Any ideas what would have caused this build breakage since release_33?<br>
><br>
>     Thanks,<br>
>     Geoffrey<br>
><br>
><br>
><br>
><br>
</div>> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
<a href="http://www.nuanti.com" target="_blank">http://www.nuanti.com</a><br>
the browser experts<br>
<br>
</font></span></blockquote></div><br></div></div>