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 590525 - g_timeout_add replacement
g_timeout_add replacement
Status: RESOLVED NOTABUG
Product: gnome-keyring
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: GNOME keyring maintainer(s)
GNOME keyring maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2009-08-02 13:34 UTC by Dominik Jastrzębski
Modified: 2009-08-06 01:56 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
the patch replaces recursive g_timeout_add calls for g_timeout_add_seconds. (1.08 KB, patch)
2009-08-02 14:30 UTC, Dominik Jastrzębski 		none Details | Review

Description Dominik Jastrzębski 2009-08-02 13:34:04 UTC 
I will work at the GTK+ g_timeout_add function replacement in gnome-keyring application. Hope the patch will be ready before September.
Comment 1 Dominik Jastrzębski 2009-08-02 14:28:25 UTC 
The patch is ready now, it was faster and easier than I expected :)
only 1 recursive call of the function in whole project.
Comment 2 Dominik Jastrzębski 2009-08-02 14:30:27 UTC 
Created attachment 139737 [details] [review]
the patch replaces recursive g_timeout_add calls for g_timeout_add_seconds.
Comment 3 Stef Walter 2009-08-06 01:56:05 UTC 
Sadly the milliseconds are actually used in this case. However this code is part of the unit tests, so it's not important to replace the call. 

Thanks for the patch and any future contributions...