From: Marko Rauhamaa <marko@pacujo.net>
To: guile-gtk@sources.redhat.com
Subject: Fixed memory leaks in gdk-1.2.defs
Date: Fri, 09 May 2003 06:33:00 -0000 [thread overview]
Message-ID: <m34r44prgb.fsf@lumo.pacujo.net> (raw)
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/
next reply other threads:[~2003-05-09 6:33 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-05-09 6:33 Marko Rauhamaa [this message]
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
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=m34r44prgb.fsf@lumo.pacujo.net \
--to=marko@pacujo.net \
--cc=guile-gtk@sources.redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).