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 580693 - Please consider using g_timeout_add_seconds to reduce wakeups
Please consider using g_timeout_add_seconds to reduce wakeups
Status: RESOLVED FIXED
Product: NetworkManager
Classification: Platform
Component: nm-applet
unspecified
Other Linux
: Normal minor
: ---
Assigned To: Dan Williams
Dan Williams
: 576144 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2009-04-29 03:12 UTC by Javier Jardón (IRC: jjardon)
Modified: 2009-10-16 04:17 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
substitute g_timeout_add by g_timeout_add_seconds (1.39 KB, patch)
2009-04-29 03:13 UTC, Javier Jardón (IRC: jjardon)
none Details | Review
substitute g_timeout_add by g_timeout_add_seconds (1.40 KB, patch)
2009-04-29 03:17 UTC, Javier Jardón (IRC: jjardon)
none Details | Review
substitute g_timeout_add by g_timeout_add_seconds v2 (1.84 KB, patch)
2009-08-10 21:44 UTC, Javier Jardón (IRC: jjardon)
none Details | Review

Description Javier Jardón (IRC: jjardon) 2009-04-29 03:12:46 UTC
From https://wiki.ubuntu.com/SavingTheWorld, (and of course according to
the gtk docs) using g_timeout_add_seconds is preferred over g_timeout_add if a
timeout in seconds is desired.
Comment 1 Javier Jardón (IRC: jjardon) 2009-04-29 03:13:43 UTC
Created attachment 133539 [details] [review]
substitute g_timeout_add by g_timeout_add_seconds
Comment 2 Javier Jardón (IRC: jjardon) 2009-04-29 03:17:53 UTC
Created attachment 133540 [details] [review]
substitute g_timeout_add by g_timeout_add_seconds

Sorry, this is the correct patch
Comment 3 Dan Williams 2009-04-30 19:24:48 UTC
*** Bug 576144 has been marked as a duplicate of this bug. ***
Comment 4 Dan Williams 2009-04-30 19:25:09 UTC
patch looks good, thanks; will commit
Comment 5 Javier Jardón (IRC: jjardon) 2009-08-10 21:44:29 UTC
Created attachment 140383 [details] [review]
substitute g_timeout_add by g_timeout_add_seconds v2

For your convenience, I sent you a new patch with the new git format

Regards
Comment 6 Dan Williams 2009-10-16 04:17:18 UTC
a0210fe4e0ff34d18e275b31dc2084c904e6d162 (master)
101b6ecb8c703c08c08a0ef8b23f7eda274f889b (0.7.x)

Thanks!