Philip Martin <> writes:

> Branko Čibej <> writes:
>> And I don't see these, either. I checked the code and we explicitly use
>> a JNI local frame in the core loop of unparseExternals. Try as I might,
>> I can't see where all those local references could be coming from.
> ImmutableList::for_each appears to be the problem.  The following ugly
> hack is not correct but does resolve the warning:

That is more complicated than necessary.  iter.has_next is not the
problem, just the function call.  Probably this is still fixing the
symptom rather than the underlying problem:

Index: subversion/bindings/javahl/native/jniwrapper/jni_list.hpp
--- subversion/bindings/javahl/native/jniwrapper/jni_list.hpp   (revision 
+++ subversion/bindings/javahl/native/jniwrapper/jni_list.hpp   (working copy)
@@ -173,7 +173,11 @@ class ImmutableList : public BaseImmutableList
       Iterator iter(get_iterator());
       while (iter.has_next())
+      {
+        m_env.PushLocalFrame(10);
         function(T(m_env, NativeT(;
+        m_env.PopLocalFrame();
+      }
       return function;


Reply via email to