Opened 18 years ago

Closed 18 years ago

#4828 closed defect (fixed)

why assign when we don't have to?

Reported by: Erik Hovland <erik@…> Owned by: Isaac Richards
Priority: minor Milestone: 0.22
Component: mythtv Version: 0.21-fixes
Severity: low Keywords:
Cc: Ticket locked: no

Description

In UIManagedTreeListType::Draw() lnode is assigned to the return value of at() and first(). But those assignments are never used. The at() and first() calls do useful work and should be done. But the assignment is unnecessary. Removing them should reduce machine code size.

Attachments (1)

libs_libmyth_uitypes.cpp-why-assign-when-unnecessary.patch​ (972 bytes ) - added by Erik Hovland <erik@…> 18 years ago.
removes assignment from at() and first() return values

Download all attachments as: .zip

Change History (3)

by Erik Hovland <erik@…>, 18 years ago

Attachment: libs_libmyth_uitypes.cpp-why-assign-when-unnecessary.patch​ added

removes assignment from at() and first() return values

comment:1 by Isaac Richards, 18 years ago

Milestone: 0.21 → 0.22

comment:2 by Nigel, 18 years ago

Resolution: → fixed
Status: new → closed

(In [16349]) Remove unwanted assignment. Patch by Erik Hovland. Closes #4828

Note: See TracTickets for help on using tickets.