Rockbox

Tasklist

FS#9557 - Correctly interpret the number parameter of the %mv wps tag

Attached to Project: Rockbox
Opened by Alexander Levin (fml2) - Sunday, 16 November 2008, 12:25 GMT
Last edited by Jonathan Gordon (jdgordon) - Saturday, 22 November 2008, 10:31 GMT
Task Type Patches
Category Applications
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

The number parameter in the %mv tag (volume changing) should be interpreted as seconds. But it's interpreted as 0.1 secs. I.e. 2.5 will wait 0.25 secs instead of 2.5. The patch fixes that.
This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Saturday, 22 November 2008, 10:31 GMT
Reason for closing:  Accepted
Comment by Alexander Levin (fml2) - Sunday, 16 November 2008, 12:37 GMT
Another try (moved private symbols from .h to .c).
Comment by Alexander Levin (fml2) - Monday, 17 November 2008, 21:53 GMT
A comment for the reviewers: it might be better to store the timeout not in 0.1 sec units (as it's done now) but directly in ticks (i.e. in 0.01 sec units). Then the interpreting code wouldn't need to do any scaling. But in the WPS structure, the subline timeout is stored as an unsigned char. That would be too small for storing ticks, hence I left it in tenth of a second. I didn't want to make the data structure bigger in size.

Loading...