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);
}
[Linux Kernel]
[Linux Cryptography]
[Fedora]
[Fedora Directory]
[Red Hat Development]