Adium

Ticket #716 (closed defect: fixed)

Opened 7 years ago

Last modified 3 years ago

performance issues with > 100 saved aways

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

Description (last modified by toby) (diff)

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

comment:1 Changed 7 years ago by toby

  • Description modified (diff)

comment:2 Changed 7 years ago by Synapse

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

Review?

comment:3 Changed 7 years ago by evands

  • priority changed from normal to high
  • Milestone set to Adium X 1.0

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 7 years ago by evands

  • Status changed from new to closed
  • Resolution set to fixed

(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 3 years ago by zacw

  • Component changed from Core Adium to Adium Core
Note: See TracTickets for help on using tickets.