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

Basic Cue sheet support #17

Closed
wants to merge 2 commits into from
Closed

Conversation

AKuHAK
Copy link
Contributor

@AKuHAK AKuHAK commented Oct 6, 2023

No description provided.

@rickgaiser
Copy link
Owner

I don't see the added value of this PR.

It will only support DVD's with 1 track of 2048 byte sectors. Normally these are just iso's. If you do have a bin/cue pair like this then you can pass the .bin file to neutrino (as if it was an .iso) becouse besides the extension these files are the same.

CD's with 1 track have some benefit, becouse they are often in cue/bin format. However, the non-2048 byte sector size means all sectors have to be memcpy'd in cdvdman_emu, plus mutiple reads and an extra bounce buffer are needed.
So doing it the bin/cue way will make it both slower and use more IOP RAM. I did a test with this that worked, but for me the downsides did not outweigh the benefit. Just convert bin/cue files to .iso, and done. (I use bchunk on linux)

CD's with multiple tracks are the only things that really need cue files (correct me if I'm wrong). But supporting this has a big impact on sofware: we need to be able to pass for instance 8 bin files to cdvdman_emu. With all the metadata to tell cdvdman_emu what to do with each file/track. This would be nice to have to get those games working, but then I looked at the list of games that really need it:
ps2_cdda_2352b_sector

It's a relatively small list with games I'm personally not interested in. So for me it was not worth it, and I stopped trying to get these games to work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants