public inbox for guile-gtk@sourceware.org
 help / color / mirror / Atom feed
* Fixed memory leaks in gdk-1.2.defs
@ 2003-05-09  6:33 Marko Rauhamaa
  2003-05-12  1:05 ` Kevin Ryde
                   ` (2 more replies)
  0 siblings, 3 replies; 9+ messages in thread
From: Marko Rauhamaa @ 2003-05-09  6:33 UTC (permalink / raw)
  To: guile-gtk

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/

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2003-06-18 23:31 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-05-09  6:33 Fixed memory leaks in gdk-1.2.defs Marko Rauhamaa
2003-05-12  1:05 ` Kevin Ryde
2003-05-14  1:18   ` Marko Rauhamaa
2003-05-12  1:08 ` Kevin Ryde
2003-05-12  7:55   ` Marko Rauhamaa
2003-05-15 23:03     ` Kevin Ryde
2003-05-16  1:02       ` Marko Rauhamaa
2003-05-17  8:08         ` Marko Rauhamaa
2003-06-18 23:31 ` GdkEvent copy under signal handler (was: Fixed memory leaks in gdk-1.2.defs) Kevin Ryde

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).