Opened 18 years ago

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

Download all attachments as: .zip

Change History (3)

by Erik Hovland <erik@…>, 18 years ago

Attachment: libs_libmythtv-missing-break.patch​ added

reworks switch/case to be a bit more efficient

comment:1 by danielk, 18 years ago

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

comment:2 by danielk, 18 years ago

Resolution: → fixed
Status: assigned → closed

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

function.

Note: See TracTickets for help on using tickets.