Opened 16 years ago

Closed 16 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@…> 16 years ago.
reworks switch/case to be a bit more efficient

Download all attachments as: .zip

Change History (3)

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

Attachment: libs_libmythtv-missing-break.patch​ added

reworks switch/case to be a bit more efficient

comment:1 Changed 16 years ago by danielk

Owner: changed from Isaac Richards to danielk
Status: new → assigned

comment:2 Changed 16 years ago by danielk

Resolution: → fixed
Status: assigned → closed

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

function.

Note: See TracTickets for help on using tickets.