Hi Will,<br><br><div class="gmail_quote">On Tue, Oct 18, 2011 at 7:56 PM, Will Dietz <span dir="ltr"><<a href="mailto:willdtz@gmail.com">willdtz@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On Tue, Oct 18, 2011 at 11:39 AM, Nicolas Geoffray<br>
<<a href="mailto:nicolas.geoffray@gmail.com">nicolas.geoffray@gmail.com</a>> wrote:<br>
> Hi Will,<br>
> That's great! Here's my first comments. I'll try to review your other<br>
> patches ASAP. Do you have commit access to the llvm repo?<br>
<br>
</div>It appears I do have commit access.  Not knowing a better way to<br>
verify if I did, I just tried committing the first three patches that<br>
you approved and it seems that it went through fine.  While the<br>
patches were approved and fairly small, hope you don't mind that I<br>
went ahead and committed them.<br></blockquote><div><br></div><div>No problem.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
On a related note, humorously enough I have commit access but the<br>
commits mailing list is grumpy :).  Should I issue some kind of<br>
subscribe request?  I'm guessing it's because I'm on the list as<br>
willdtz@gmail, and commit as wdietz2@uiuc.edu...<br></blockquote><div><br></div><div>Coud you try to be on the list as @uiuc? It'd be annoying not to see your changes in the ML.</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>
> patches 1, 2, 3: look good!<br>
> patch 4: looking at the code, I realize that NATIVE_JNI is completely bogus<br>
> today, so you can safely remove the use of the macro and the arguments it<br>
> surrounds. Also, even though it's just casts, you should not use jobject and<br>
> jclass in these methods. Just use JavaObject. Also, please remove the //<br>
> printf.<br>
<br>
</div>Okay, sure thing.  If NATIVE_JNI is bogus, I'll put together a patch<br>
ripping it out at some point, if you don't beat me to it :).<br></blockquote><div><br></div><div>That'd be nice. Thanks!</div><div><br></div><div>Nicolas</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<br>
Sorry about the debugging printf, whoops.<br>
<br>
> patch 5: looks good!<br>
<br>
Great! :)<br>
<br>
Thanks for the quick response and your comments!<br>
<font color="#888888"><br>
~Will<br>
</font></blockquote></div><br>