Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#10764 closed Patch - Bug Fix (fixed)

Correct unhandled enumeration value in videoouttypes.h - detected by clang

Reported by: Gary Buhrmaster <gary.buhrmaster@…> Owned by: beirdo
Priority: minor Milestone: 0.26
Component: MythTV - Video Library Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Eliminate a pedantic compiler warning from clang in videooutypes.h. Clang will warn when a switch statement does not explicitly handle all cases. This patch adds in the missing case statement (with others that simply break).

Attachments (1)

videoouttypes.h.patch​ (538 bytes) - added by Gary Buhrmaster <gary.buhrmaster@…> 8 years ago.

Download all attachments as: .zip

Change History (4)

Changed 8 years ago by Gary Buhrmaster <gary.buhrmaster@…>

Attachment: videoouttypes.h.patch​ added

comment:1 Changed 8 years ago by Gary Buhrmaster <gary.buhrmaster@…>

Owner: set to Gary Buhrmaster <gary.buhrmaster@…>
Resolution: → fixed
Status: new → closed

In e2a0efd88ce48698451a36fe9c15b390e7fdd73d/mythtv:

Add missing case statement

Also added default. This file needs to have its formatting fixed to match
MythTV standards. The switch statements in particular have wonky indentations
and break; and such should be on separate lines. A task for later.

Fixes #10764

Signed-off-by: Gavin Hurlbut <ghurlbut@…>

comment:2 Changed 8 years ago by beirdo

Owner: changed from Gary Buhrmaster <gary.buhrmaster@…> to beirdo

comment:3 Changed 8 years ago by Raymond Wagner

Milestone: unknown → 0.26
Note: See TracTickets for help on using tickets.