Notice: A non well formed numeric value encountered in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 96 Notice: A non well formed numeric value encountered in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 96 Notice: A non well formed numeric value encountered in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 96 Deprecated: Function create_function() is deprecated in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 104 Deprecated: The each() function is deprecated. This message will be suppressed on further calls in /sites/rockbox.org/flyspray/adodb/adodb.inc.php on line 845 Deprecated: Function create_function() is deprecated in /sites/rockbox.org/flyspray/includes/class.user.php on line 111 FS#11828 : Fix core mod metadata parser

Rockbox

Tasklist

FS#11828 - Fix core mod metadata parser

Attached to Project: Rockbox
Opened by Thomas Martitz (kugel.) - Thursday, 23 December 2010, 19:35 GMT
Last edited by Thomas Martitz (kugel.) - Friday, 21 January 2011, 20:09 GMT
Task Type Patches
Category ID3 / meta data
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Release 3.7.1
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

The mod metadata parser checks if the file in question is a mod file in a very poor manner: not at all. This is a problem when building the database on a PC, because any file that has .mod is accepted, including those text files generated during Linux kernel compilation.

Based on MikiMod, I made this patch to fix it. Please let me know if it rejects any *valid* mod file. It should now accept the same files that the MikMod plugin accepts.
This task depends upon

Closed by  Thomas Martitz (kugel.)
Friday, 21 January 2011, 20:09 GMT
Reason for closing:  Accepted
Additional comments about closing:  r29105
Comment by Jason Yu (captainkewl) - Wednesday, 05 January 2011, 04:39 GMT
Looks fine. Invalid .mod files are correctly ignored during database generation and skipped over if they make it into a playlist. Only thing that sticks out to me is that Mikmod is spelled wrong.
Comment by MichaelGiacomelli (saratoga) - Friday, 21 January 2011, 00:38 GMT
It would be nice to get this committed so that the database is a little more stable.

Loading...