Hi, Sorry to nag about this, but it would be good to get this applied. Without it, the Swing Demo leaks like a sieve on JamVM :) Rob. ---------- Forwarded message ---------- From: Robert Lougher <rob.lougher@xxxxxxxxx> Date: Fri, Oct 10, 2008 at 7:55 AM Subject: Missing ReleaseArrayElements To: classpath-patches <classpath-patches@xxxxxxx> Hi, This fixes a missing ReleaseArrayElements in the GTK peer code (fixing a noticeable memory leak). Rob.
Index: native/jni/gtk-peer/gnu_java_awt_peer_gtk_CairoGraphics2D.c =================================================================== RCS file: /sources/classpath/classpath/native/jni/gtk-peer/gnu_java_awt_peer_gtk_CairoGraphics2D.c,v retrieving revision 1.26 diff -u -r1.26 gnu_java_awt_peer_gtk_CairoGraphics2D.c --- native/jni/gtk-peer/gnu_java_awt_peer_gtk_CairoGraphics2D.c 11 Sep 2007 09:48:49 -0000 1.26 +++ native/jni/gtk-peer/gnu_java_awt_peer_gtk_CairoGraphics2D.c 10 Oct 2008 06:44:01 -0000 @@ -372,6 +372,7 @@ } gdk_threads_leave(); + (*env)->ReleaseLongArrayElements (env, java_fontset, fonts, 0); g_free(glyphs); }