Opened 17 years ago

Closed 16 years ago

Last modified 15 years ago

#6340 closed enhancement (fixed)

Replace QMap with QHash where possible

Reported by: Matthias "mortalmatt" Dahl <devel@…> Owned by: Isaac Richards
Priority: trivial Milestone: unknown
Component: MythTV - General Version: head
Severity: low Keywords:
Cc: Ticket locked: yes

Description

Since QMap is rather expensive for lookups but also inserts, we should use QHash where we don't rely on a sorted vector.

Attachments (2)

DVBStreamHandler_replaceQMap.diff​ (4.9 KB ) - added by Matthias "mortalmatt" Dahl <devel@…> 17 years ago.
MPEGStreamData_replaceQMap.diff​ (5.6 KB ) - added by Matthias "mortalmatt" Dahl <devel@…> 17 years ago.

Download all attachments as: .zip

Change History (7)

by Matthias "mortalmatt" Dahl <devel@…>, 17 years ago

Attachment: DVBStreamHandler_replaceQMap.diff​ added

by Matthias "mortalmatt" Dahl <devel@…>, 17 years ago

Attachment: MPEGStreamData_replaceQMap.diff​ added

comment:1 by anonymous, 17 years ago

The former two patches reduce the time spent in related Qt functions by approx. 50% during recording.

comment:2 by danielk, 17 years ago

Milestone: 0.22 → unknown
Priority: minor → trivial
Severity: medium → low

comment:3 by shane, 16 years ago

(In [20825]) Refs #6340. Convert settings cache from QMap to QHash for faster lookups.

comment:4 by danielk, 16 years ago

Resolution: → fixed
Status: new → closed

comment:5 by stuartm, 15 years ago

Ticket locked: set
Note: See TracTickets for help on using tickets.