public inbox for guile-gtk@sourceware.org
 help / color / mirror / Atom feed
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/

             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).