Rockbox

Tasklist

FS#6646 - Speex support breaks prev WPS %fc

Attached to Project: Rockbox
Opened by Ben Keroack (bk) - Sunday, 11 February 2007, 21:57 GMT
Task Type Bugs
Category Themes
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

When adding support for new codecs, the new codec should always be added to the *END* of the swcodec section of the AFMT enum (firmware/export/id3.h). This preserves the accuracy of WPSs that use the %fc tag to display the currently playing codec (when using the new codec the tag will default to 'unknown').

When speex support was added it was put beween SID and ADX in the AFMT enum, breaking all WPSs that use %fc. Attached is a patch fixing this.
This task depends upon

Closed by  Dave Chapman (linuxstb)
Sunday, 11 February 2007, 22:33 GMT
Reason for closing:  Accepted
Additional comments about closing:  Committed, thanks.

Loading...