[Pidgin] #8831: Patch for batching multi-second timeouts using purple_timeout_add_seconds in pidgin/finch

Pidgin trac at pidgin.im
Wed Apr 1 03:47:21 EDT 2009


#8831: Patch for batching multi-second timeouts using purple_timeout_add_seconds
in pidgin/finch
------------------------------------+---------------------------------------
 Reporter:  arunanbala              |           Owner:  sadrul             
     Type:  patch                   |          Status:  new                
Milestone:  Patches Needing Review  |       Component:  finch (gnt/ncurses)
  Version:  2.5.5                   |      Resolution:                     
 Keywords:                          |   Launchpad_bug:                     
------------------------------------+---------------------------------------
Description changed by arunanbala:

Old description:

> This is a patch for Finch to batch some timeouts, described in
> https://wiki.ubuntu.com/SavingTheWorld and previous changes to
> eventloop.c in Pidgin (noted in chagelog for release 2.1.0).
>
> I did not change a few possible cases in finch/libgnt as it seems that
> libgnt is not meant to be libpurple dependent - is that the case? If so,
> would #ifdeffing for glib 2.14 be OK in those cases?

New description:

 This is a patch for Finch to batch some timeouts, described in
 https://wiki.ubuntu.com/SavingTheWorld and previous changes to eventloop.c
 in Pidgin (noted in chagelog for release 2.1.0).

 Using purple_timeout_add_seconds allows for multiple timers to be fired at
 once, allowig CPUs to sleep longer and increasing power efficiency.

 I did not change a few possible cases in finch/libgnt as it seems that
 libgnt is not meant to be libpurple dependent - is that the case? If so,
 would #ifdeffing for glib 2.14 be OK in those cases?

--

-- 
Ticket URL: <http://developer.pidgin.im/ticket/8831#comment:3>
Pidgin <http://pidgin.im>
Pidgin


More information about the Tracker mailing list