Opened 17 years ago
Closed 17 years ago
#4828 closed defect (fixed)
why assign when we don't have to?
Reported by: | 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)
Change History (3)
Changed 17 years ago by
Attachment: | libs_libmyth_uitypes.cpp-why-assign-when-unnecessary.patch added |
---|
comment:1 Changed 17 years ago by
Milestone: | 0.21 → 0.22 |
---|
Note: See
TracTickets for help on using
tickets.
removes assignment from at() and first() return values