Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#7831 - FLAC files with "unknown" total samplecount are rejected

Attached to Project: Rockbox
Opened by Dave Chapman (linuxstb) - Tuesday, 25 September 2007, 10:48 GMT+1
Last edited by Paul Louden (Llorean) - Friday, 12 October 2007, 06:02 GMT+1
Task Type Patches
Category Codecs
Status New
Assigned To No-one
Player type All players
Severity Low
Priority Normal
Reported Version current build
Due in Version Undecided
Due Date Undecided
Percent Complete 0%
Private No

Details

The current Rockbox FLAC codec rejects files (in apps/metadata/flac.c) where the "samplecount" field in the header is 0. However, the FLAC specification says that this is a valid value and should be interpreted as "unknown".

The only encoder I've seen which generates such files is ffmpeg.
This task depends upon

Comment by Joshua Hulst (jhulst) - Friday, 12 October 2007, 05:44 GMT+1
When totalsamples (samplerate) is 0, length and bitrate are set to 0 because their calculation depends on samplerate, else behave as before. Setting the length to 0 removes the possibility for seeking during playback, but the file is still loadable and playable.

This is my first patch, all comments and suggestions are appreciated.

Josh
   samplecount.diff (1.2 KiB)
Comment by Rosso Maltese (asettico) - Saturday, 01 March 2008, 12:41 GMT+1
Just some slight modifications to apply to current build (16460) without fuzz factor.
   samplecount-v2.patch (1.2 KiB)

Loading...