Opened 15 years ago

Closed 14 years ago

#7169 closed defect (fixed)

Passing RecordingInfo objects by value is expensive

Reported by: Erik Hovland <erik@…> Owned by: paulh
Priority: trivial Milestone: 0.23
Component: MythTV - General Version: head
Severity: low Keywords:
Cc: Ticket locked: no

Description

RecordingInfo? objects are large. So passing them by value is expensive. The member functions ShowRecordingDialog? and ShowNotRecordingDialog? in ScheduleCommon? do this, but they can be rewritten to pass by reference.

Attachments (1)

programs_mythfrontend-pass-by-value​ (2.6 KB) - added by Erik Hovland <erik@…> 15 years ago.
Pass RecordingInfo? by reference to use fewer resources.

Download all attachments as: .zip

Change History (3)

Changed 15 years ago by Erik Hovland <erik@…>

Attachment: programs_mythfrontend-pass-by-value​ added

Pass RecordingInfo? by reference to use fewer resources.

comment:1 Changed 14 years ago by paulh

Milestone: unknown → 0.23
Owner: changed from Isaac Richards to paulh
Status: new → assigned

comment:2 Changed 14 years ago by paulh

Resolution: → fixed
Status: assigned → closed

(In [23270]) Pass by reference the RecordingInfo? in a few places in ScheduleCommon?. Patch from Erik Hovland. Closes #7169.

Note: See TracTickets for help on using tickets.