Opened 13 years ago

Closed 13 years ago

#11581 closed Patch - Bug Fix (fixed)

Patch to remove redundant (logically useless) check in decoderbase.cpp

Reported by: Gary Buhrmaster <gary.buhrmaster@…> Owned by: Jim Stichnoth
Priority: minor Milestone: 0.27
Component: MythTV - General Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Static analysis (IEAEYEBALL), during the analysis of coverity 700308 which resulted in ticket 11580, noticed that the check in a number of conditionals for numStreams not being zero was redundant, since the function checked for that value near the beginning, and returned false if it was zero. Eliminate the extra checks.

Patch: ​https://github.com/garybuhrmaster/mythtv/commit/18b4ce0e1107257748060bca754a9dd69878e8fe

Change History (2)

comment:1 by stuartm, 13 years ago

Milestone: unknown → 0.27
Owner: set to Jim Stichnoth
Status: new → assigned

comment:2 by Gary Buhrmaster <gary.buhrmaster@…>, 13 years ago

Resolution: → fixed
Status: assigned → closed

In 431669005cd87124a199da013b59bf29a9bc58e9/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available
Note: See TracTickets for help on using tickets.