You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ph...@apache.org on 2018/04/07 19:20:49 UTC
svn commit: r1828613 -
/subversion/trunk/subversion/bindings/javahl/native/jniwrapper/jni_list.hpp
Author: philip
Date: Sat Apr 7 19:20:48 2018
New Revision: 1828613
URL: http://svn.apache.org/viewvc?rev=1828613&view=rev
Log:
Fix a local refs capacity warning when unparsing externals.
* subversion/bindings/javahl/native/jniwrapper/jni_list.hpp
(ImmutableList::for_each): Use a local frame to avoid acccumulating
local refs from constructors.
Modified:
subversion/trunk/subversion/bindings/javahl/native/jniwrapper/jni_list.hpp
Modified: subversion/trunk/subversion/bindings/javahl/native/jniwrapper/jni_list.hpp
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/bindings/javahl/native/jniwrapper/jni_list.hpp?rev=1828613&r1=1828612&r2=1828613&view=diff
==============================================================================
--- subversion/trunk/subversion/bindings/javahl/native/jniwrapper/jni_list.hpp (original)
+++ subversion/trunk/subversion/bindings/javahl/native/jniwrapper/jni_list.hpp Sat Apr 7 19:20:48 2018
@@ -173,7 +173,10 @@ public:
{
Iterator iter(get_iterator());
while (iter.has_next())
- function(T(m_env, NativeT(iter.next())));
+ {
+ ::Java::LocalFrame frame(m_env);
+ function(T(m_env, NativeT(iter.next())));
+ }
return function;
}
};