AuroraMiddleware/flac
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

cuesheet.c: Allow quotes around ISRC field

Browse Source 
With this change, flac now accepts cuesheets where the ISRC field is
enclosed in double quotes. Added a test for this as well.

Closes: https://sourceforge.net/p/flac/bugs/436/
This commit is contained in:
Erik de Castro Lopo 2016-04-26 17:55:36 +10:00
parent 50af1bcf63
commit 3de7413a36
3 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/share/grabbag/cuesheet.c
View File

@ -404,7 +404,7 @@ static FLAC__bool local__cuesheet_parse_(FILE *file, const char **error_message,
*error_message = "ISRC command must come after TRACK but before INDEX";
return false;
}
if(0 == (field = local__get_field_(&line, /*allow_quotes=*/false))) {
if(0 == (field = local__get_field_(&line, /*allow_quotes=*/true))) {
*error_message = "ISRC is missing ISRC number";
return false;
}

1
test/cuesheet.ok
View File

@ -91,3 +91,4 @@ POSITIVE cuesheets/good.001.cue
POSITIVE cuesheets/good.002.dos_format.cue
POSITIVE cuesheets/good.003.missing_final_newline.cue
POSITIVE cuesheets/good.004.dos_format.missing_final_newline.cue
POSITIVE cuesheets/good.005.quoted.isrc.cue

6
test/cuesheets/good.005.quoted.isrc.cue Normal file
View File

@ -0,0 +1,6 @@
TRACK 01 AUDIO
TITLE "Foo"
PERFORMER "Bar"
DISC_ID ""
ISRC "US-SM1-23-45678"
INDEX 01 00:00:00