Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Order of TITLE and INDEX in generated cue sheet. #225

Open
pasaran opened this issue Apr 4, 2024 · 1 comment
Open

Order of TITLE and INDEX in generated cue sheet. #225

pasaran opened this issue Apr 4, 2024 · 1 comment
Labels
done The task is done, but not yet in the release.

Comments

@pasaran
Copy link

pasaran commented Apr 4, 2024

I'm splitting big flac file into several tracks with option "Create per track CUE sheet".
This is an example of generated cue sheet:

PERFORMER "The Beatles"
TITLE "Abbey Road"

FILE "01 - Come Together.flac" WAVE
  TRACK 01 AUDIO
    INDEX 01 00:00:00
    TITLE "Come Together"

FILE "02 - Something.flac" WAVE
  TRACK 02 AUDIO
    INDEX 01 00:00:00
    TITLE "Something"

...

Then I open this cue sheet in Cog player and first track doesn't have a title. Second track has title of the first track etc.
If I manually change order of INDEX and TITLE everything is fine.

I tried to find a specification of cue format. There are many of them and not one of them describe the order of commands in cue file. But all examples always have INDEX as the last command in TRACK section.
And then I found this document: INDEX command — it's very detailed and it describes order of commands. According to it INDEX should be after TITLE. I've no idea is true or not tbh.

So maybe it's bug in Cog — some other players play this cue sheet just fine.
Or maybe it's a bug in Flacon and Cog just uses more strict spec of cue sheets than other players.
I don't really know.

@SokoloffA
Copy link
Member

Thanks for digging deep into the problem!

CUE looks simple, but it's quite a confusing thing. There is also a non-trivial dependence between FILE and INDEX tags.

I changed the tag order, now the tag order corresponds to the CUE file created by CUETools. I hope my changes don't break issue #172. I need some time to test non-trivial situations.

@SokoloffA SokoloffA added the done The task is done, but not yet in the release. label Apr 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
done The task is done, but not yet in the release.
Projects
None yet
Development

No branches or pull requests

2 participants