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 573921 - Please consider using g_timeout_add_seconds to reduce wakeups
Please consider using g_timeout_add_seconds to reduce wakeups
Status: RESOLVED FIXED
Product: dia
Classification: Other
Component: general
devel
Other All
: Normal enhancement
: 0.97
Assigned To: Dia maintainers
Dia maintainers
Depends on:
Blocks:
 
 
Reported: 2009-03-03 14:04 UTC by Arunan Balasubramaniam
Modified: 2009-04-11 14:27 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Patch for when glib 2.6 only is required (as at present) (511 bytes, patch)
2009-03-03 14:06 UTC, Arunan Balasubramaniam 		committed Details | Review
Patch that makes dia require glib 2.14 (1012 bytes, patch)
2009-03-03 14:07 UTC, Arunan Balasubramaniam 		reviewed Details | Review

Description Arunan Balasubramaniam 2009-03-03 14:04:51 UTC 
From https://wiki.ubuntu.com/SavingTheWorld , two versions of a patch to change the autosave timer to use g_timeout_add_seconds().

This patch does not change dynamic_refresh.c line 92 as I belive that uses ms resolution (or finer than 1s anyway); is this correct?
Comment 1 Arunan Balasubramaniam 2009-03-03 14:06:57 UTC 
Created attachment 129941 [details] [review]
Patch for when glib 2.6 only is required (as at present)

Version of the patch that checks for glib 2.14
Comment 2 Arunan Balasubramaniam 2009-03-03 14:07:54 UTC 
Created attachment 129942 [details] [review]
Patch that makes dia require glib 2.14

Alternative patch with no #ifdef but ups the glib requirement to 2.14
Comment 3 Hans Breuer 2009-04-11 14:27:43 UTC 
Thanks, applied the first version:

2009-04-11  Hans Breuer  <hans@breuer.org>

	* app/app_procs.c :  patch from Arunan Balasubramaniam, bug #573921 to
	to reduce wakeups prefer g_timeout_add_seconds() over g_timeout_add()