After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 577048 - GLib.Timeout.add_seconds_full and GLib.Source.set_callback require DestroyNotify to be passed manually
GLib.Timeout.add_seconds_full and GLib.Source.set_callback require DestroyNot...
Status: RESOLVED FIXED
Product: vala
Classification: Core
Component: Bindings
0.5.x
Other Linux
: Normal trivial
: ---
Assigned To: Jürg Billeter
Vala maintainers
Depends on:
Blocks:
 
 
Reported: 2009-03-28 07:52 UTC by Evan Nemerson
Modified: 2009-03-29 19:16 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Make GLib.Timeout.add_seconds_full and GLib.Source.set_callback generate the appropriate GDestroyNotify arguments automatically. Fixes bug 577048. (1.46 KB, patch)
2009-03-28 07:55 UTC, Evan Nemerson
committed Details | Review

Description Evan Nemerson 2009-03-28 07:52:07 UTC
They use SourceFunc func, DestroyNotify? notify instead of SourceFunc# func.
Comment 1 Evan Nemerson 2009-03-28 07:55:48 UTC
Created attachment 131540 [details] [review]
Make GLib.Timeout.add_seconds_full and GLib.Source.set_callback generate the appropriate GDestroyNotify arguments automatically. Fixes bug 577048.
Comment 2 Jürg Billeter 2009-03-29 19:16:53 UTC
commit e2e7652dfd9bb82f0f51e4c7036bec3d0838fc40
Author: Evan Nemerson <evan@polussystems.com>
Date:   Sat Mar 28 00:54:10 2009 -0700

    glib-2.0: Fix g_timeout_add_seconds_full and g_source_set_callback
    
    Make GLib.Timeout.add_seconds_full and GLib.Source.set_callback
    generate the appropriate GDestroyNotify arguments automatically.
    
    Fixes bug 577048.