Looks good!<div><br></div><div>Alternatively, we could allocate space up front in the method, but I don't think that's worth the trouble.</div><div><br></div><div>Nicolas<br><br><div class="gmail_quote">On Tue, Nov 1, 2011 at 5:53 AM, Will Dietz <span dir="ltr"><<a href="mailto:wdietz2@illinois.edu">wdietz2@illinois.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Inlined below.<br>
<br>
The idea here is simply that we'll more or less always have capacity,<br>
and since we dynamically allocate new blocks of references (100 at a<br>
time) there's no meaningful way to check if we're at capacity.<br>
<br>
Thoughts welcome of course :).<br>
<br>
~Will<br>
<br>
>From b40f02d173d5aeff6b5e68522ac5d8342f58c2c0 Mon Sep 17 00:00:00 2001<br>
From: Will Dietz <<a href="mailto:w@wdtz.org">w@wdtz.org</a>><br>
Date: Mon, 31 Oct 2011 16:36:17 -0500<br>
Subject: [PATCH 5/8] Don't abort in JNI EnsureLocalCapacity, assume we have<br>
 capacity.<br>
<br>
---<br>
 lib/J3/VMCore/Jni.cpp |    3 +--<br>
 1 files changed, 1 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/lib/J3/VMCore/Jni.cpp b/lib/J3/VMCore/Jni.cpp<br>
index 8f5a19f..6fceac6 100644<br>
--- a/lib/J3/VMCore/Jni.cpp<br>
+++ b/lib/J3/VMCore/Jni.cpp<br>
@@ -297,8 +297,7 @@ jobject NewLocalRef(JNIEnv *env, jobject ref) {<br>
<br>
<br>
 jint EnsureLocalCapacity(JNIEnv* env, jint capacity) {<br>
-  NYI();<br>
-  abort();<br>
+  // Assume we have capacity<br>
   return 0;<br>
 }<br>
<font color="#888888"><br>
--<br>
1.7.5.1<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>
</font></blockquote></div><br></div>