Opened 11 years ago

Closed 11 years ago

#5687 closed defect (fixed)

switch/case in calcBPM is a touch strange

Reported by: Erik Hovland <erik@…> Owned by: danielk
Priority: trivial Milestone: unknown
Component: mythtv Version: head
Severity: low Keywords:
Cc: Ticket locked: no

Description

The switch/case in calcBPM in util-xvmc.h is a bit strange. Almost all of the cases produce 6 as the return value. So they could be consolidated to just two decisions.

Attachments (1)

libs_libmythtv-missing-break.patch (947 bytes) - added by Erik Hovland <erik@…> 11 years ago.
reworks switch/case to be a bit more efficient

Download all attachments as: .zip

Change History (3)

Changed 11 years ago by Erik Hovland <erik@…>

reworks switch/case to be a bit more efficient

comment:1 Changed 11 years ago by danielk

Owner: changed from Isaac Richards to danielk
Status: newassigned

comment:2 Changed 11 years ago by danielk

Resolution: fixed
Status: assignedclosed

(In [18656]) Fixes #5687. Add missing break statements in calcBPM

function.

Note: See TracTickets for help on using tickets.