Fwd: Missing ReleaseArrayElements

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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);
 }
 

[Index of Archives]     [Linux Kernel]     [Linux Cryptography]     [Fedora]     [Fedora Directory]     [Red Hat Development]

  Powered by Linux