Opened 16 years ago
Closed 16 years ago
#4826 closed defect (invalid)
remove useless check of mytcontextprivate pointer 'd'
Reported by: | Owned by: | Isaac Richards | |
---|---|---|---|
Priority: | minor | Milestone: | 0.21 |
Component: | mythtv | Version: | 0.21-fixes |
Severity: | low | Keywords: | |
Cc: | Ticket locked: | no |
Description
In ticket 4745, Isaac correctly pointed out that it is impossible for a mythcontext object to have a null pointer d since d is new'ed in the constructor (and it would exception out if the new didn't work). That means that the checks for 'd' being a null pointer in MythContext::GetSetting?() are useless and should be removed before they confuse anybody.
Attachments (1)
Change History (2)
Changed 16 years ago by
Attachment: | libs_libmyth_mythcontext.cpp-d-check-is-useless.patch added |
---|
comment:1 Changed 16 years ago by
Resolution: | → invalid |
---|---|
Status: | new → closed |
This was already done in trunk, and doesn't need to be in the branch.
Note: See
TracTickets for help on using
tickets.
removes useless d null pointer checks