Adium

Opened 15 years ago

Closed 15 years ago

Last modified 11 years ago

#716 closed defect (fixed)

performance issues with > 100 saved aways

Reported by: toby Owned by: anybody
Milestone: Adium X 1.0 Component: Adium Core
Version: Severity: normal
Keywords: Cc:
Patch Status:

Description (last modified by toby)

If you have a lot (114 in the observed case), it takes several seconds for preferences to save when adding new away messages.

Change History (5)

comment:1 Changed 15 years ago by toby

Description: modified (diff)

comment:2 Changed 15 years ago by Synapse

Should we be catering to insane users? What on earth do you need that many for?

Review?

comment:3 Changed 15 years ago by Evan Schoenberg

Milestone: Adium X 1.0
priority: normalhigh

It really shouldn't be that expensive to make a one-item change to the menu... right now it rebuilds many times (for per-account status, etc) immediately, I believe. The whole thing is a bit broken in .9svn and needs to be reviewed.

comment:4 Changed 15 years ago by Evan Schoenberg

Resolution: fixed
Status: newclosed

(In [13941]) * Fixed updating of checkmarks / dashes to reflex the current status

  • Using -[AIStatus menuIcon] to fix recent problems with icons flipping
  • This and the previous commit to this file should improve performance when switching statuses or adding statuses. Fixes #716 (Please reopen #716 with a sample if performance continues to be a problem with large numbers of saved statuses).

comment:5 Changed 11 years ago by Zachary West

Component: Core AdiumAdium Core
Note: See TracTickets for help on using tickets.