This is the mail archive of the
guile-gtk@sources.redhat.com
mailing list for the Guile project.
Fixed memory leaks in gdk-1.2.defs
- From: Marko Rauhamaa <marko at pacujo dot net>
- To: guile-gtk at sources dot redhat dot com
- Date: 08 May 2003 23:25:08 -0700
- Subject: Fixed memory leaks in gdk-1.2.defs
Summary: Fixed memory leaks in gdk-1.2.defs.
Details:
- build-guile-gtk-1.2, guile-gtk.c: GDK and GTK differ in the reference
count policy. While GTK returns the widgets with a reference count 0,
GDK returns them with a reference count 1. That means that
gdk-1.2.defs should not increment the reference count -- but it was
doing that.
It turns out that the different GDK types don't need a copy option (=
"method") at all whether they have reference counts or not. I made
the copy option optional (previously the omission of the copy option
caused a runtime crash).
- gdk-1.2.defs: Added GdkFontType. Added it to GdkFont.
- gdk-1.2.defs: Moved GdkFillRule among other enums.
- gdk-1.2.defs: Removed copy options (previously: gdk_..._ref,
gtk_no_copy, gtk_fake_copy, gdk_..._copy).
- gdk-1.2.defs: Removed now redundant (copy #f) options from return
values.
- gdk-1.2.defs: Removed redundant conversion option from GdkFont.
- gdk-1.2.defs: Changed GdkWChar from uint to uint32 (since it is now
available).
- gdk-1.2.defs: Removed useless size options -- but left it with
gtimer, since define-struct still requires a dummy size.
- gdk-1.2.defs: Indicated that gdk_font_intern and gdk_color_intern are
now deprecated. (I don't think anybody used them anyway.)
- gdk-1.2.defs, gdk-support.c, guile-gtk.h: Changed the signature of
pixel-from-rgb from (visual red green blue) to (visual rgb) for
consistency's sake.
- gdk-support.c, gtk-compat.c: Now that copying is no longer done,
needed to explicitly copy the return values of gdk-color-new,
gdk-color-parse and gtk-style-get-white.
- gdk-support.c, guile-gtk.h: Removed sgtk_gdk_drag_context_ref as
no longer needed.
Marko
--
Marko Rauhamaa mailto:marko@pacujo.net http://pacujo.net/marko/