[prev in list] [next in list] [prev in thread] [next in thread]
List: flac-dev
Subject: [Flac-dev] Fix cuesheet.c to allow metaflac_test.sh to run to
From: Earl Chew <earl_chew () yahoo ! com>
Date: 2012-01-06 17:00:17
Message-ID: 1325869217.50138.YahooMailNeo () web36504 ! mail ! mud ! yahoo ! com
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
When reading the INDEX from the cue sheet, the format MM:SS:FF format is disallowed \
if the sample frequency is not a multiple of 75 because the index would only be \
approximate. However, 00:00:00 is _exact_ because it denotes the start of the track, \
so allow it as a special case. This allows metaflac_test.sh to pass.
[Attachment #5 (text/html)]
<html><body><div style="color:#000; background-color:#fff; font-family:times new \
roman, new york, times, serif;font-size:12pt"><div>When reading the INDEX from the \
cue sheet, the format MM:SS:FF format is disallowed if the sample frequency is not a \
multiple of 75 because the index would only be approximate. However, 00:00:00 is \
_exact_ because it denotes the start of the track, so allow it as a special case. \
This allows metaflac_test.sh to \
pass.<br><br></div></div></body></html>
---2114655128-51558577-1325869217=:50138--
["flac.diff" (application/octet-stream)]
diff --git a/src/share/grabbag/cuesheet.c b/src/share/grabbag/cuesheet.c
index 682ee3d..1f98c15 100644
--- a/src/share/grabbag/cuesheet.c
+++ b/src/share/grabbag/cuesheet.c
@@ -366,7 +366,8 @@ static FLAC__bool local__cuesheet_parse_(FILE *file, const char \
**error_message, }
}
}
- else if(sample_rate % 75) {
+ else if(sample_rate % 75 && xx) {
+ /* only sample zero is exact */
*error_message = "illegal INDEX offset (MM:SS:FF form not allowed if sample \
rate is not a multiple of 75)"; return false;
}
_______________________________________________
Flac-dev mailing list
Flac-dev@xiph.org
http://lists.xiph.org/mailman/listinfo/flac-dev
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic