Adium

Opened 10 years ago

Closed 8 years ago

#13932 closed enhancement (fixed)

Consider using the libdispatch event loop instead of CFRunLoop in adiumPurpleEventLoop.m

Reported by: Catfish_Man Owned by: earthmkii
Milestone: Adium 1.5 Component: Adium Core
Version: 1.5hg Severity: normal
Keywords: hotness Cc:
Patch Status:

Description

dispatch sources and timers instead of CF ones :)

Change History (7)

comment:1 follow-up: Changed 10 years ago by am

Will have to wait until 10.7 is released I guess.

comment:2 in reply to: ↑ 1 Changed 10 years ago by earthmkii

Replying to am:

Will have to wait until 10.7 is released I guess.

Why? 1.4 already requires 10.5, and > 50% of our user base are already on some version of 10.6

by the time 1.5 is ready for release, I don't think this will be an issue.

comment:3 Changed 10 years ago by am

yes, because by then, 10.8 will be out ;)

comment:4 Changed 10 years ago by earthmkii

  • Owner set to earthmkii
  • Status changed from new to assigned

comment:5 Changed 9 years ago by Robby

  • Milestone set to Good idea for "later"

comment:6 Changed 9 years ago by sphynx

Been working on this here: https://bitbucket.org/xnyhps/adium_dispatch

The purple-eventloop stuff is all done, I think. I'm now also looking at other places where libdispatch can be used.

comment:7 Changed 8 years ago by sphynx

  • Milestone changed from Good ideas for later to Adium 1.5
  • Resolution set to fixed
  • Status changed from assigned to closed

This was merged into trunk in af2c953c613e.

adium_timeout_add_seconds was added as well, so we can hopefully save a couple of µW. :)

Note: See TracTickets for help on using tickets.