Hi Will,<div><br></div><div>That's great! Here's my first comments. I'll try to review your other patches ASAP. Do you have commit access to the llvm repo?</div><div><br></div><div>patches 1, 2, 3: look good!</div>
<div><br></div><div>patch 4: looking at the code, I realize that NATIVE_JNI is completely bogus today, so you can safely remove the use of the macro and the arguments it surrounds. Also, even though it's just casts, you should not use jobject and jclass in these methods. Just use JavaObject. Also, please remove the // printf.</div>
<div><br></div><div>patch 5: looks good!</div><div><br><div class="gmail_quote">On Tue, Oct 18, 2011 at 12:41 AM, 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;">Hi,<br>
<br>
Attached are various patches that I've refactored out from the OpenJDK<br>
port, as they aren't necessarily part of switching runtimes and help<br>
the Classpath implementation too.<br>
<br>
Hopefully OpenJDK patches will be coming in the semi-near future...<br>
<br>
Please let me know if you have any questions or comments, I'm rather<br>
happy to discuss :).<br>
<br>
Thanks, and have a good one!<br>
<font color="#888888"><br>
~Will<br>
</font><br>_______________________________________________<br>
vmkit-commits mailing list<br>
<a href="mailto:vmkit-commits@cs.uiuc.edu">vmkit-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/vmkit-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/vmkit-commits</a><br>
<br></blockquote></div><br></div>