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

DSK limited to 80 sectors #101

Open
TFHFony opened this issue Dec 18, 2023 · 2 comments
Open

DSK limited to 80 sectors #101

TFHFony opened this issue Dec 18, 2023 · 2 comments

Comments

@TFHFony
Copy link

TFHFony commented Dec 18, 2023

.DSK emulation is limited to 720Kbyte / 80 sectors. Non-Standard disks that use the 81st or 82nd sector cannot be loaded as a disk image.
Example: https://download.file-hunter.com/Games/MSX2/DSK/Dummieland%20(1994)(Soksoft)(Disk%201%20of%202).zip

The use of extra sectors is quite common, also as a copy-protection.
I've also requested the support of .DMK images (#66) which could help running this kind of disks.

@NullPointerReference
Copy link

NullPointerReference commented Mar 8, 2024

This is possibly a quick-win; by changing:

this.MEDIA_TYPE_VALID_SIZES = [ 737280, 655360, 368640, 327680, 184320, 163840 ]; // All supported floppy formats

to: this.MEDIA_TYPE_VALID_SIZES = [ 755712, 746496, 737280, 655360, 377856, 373248, 368640, 327680, 184320, 163840]

I.e. adding the length for double-sided, 82, 81 tracks and single-sided 82 and 81 disk-image sizes (in that) order.
This seems to work with the provided example file

@NullPointerReference
Copy link

PS: Also tested this on file-hunter.com (thanks @TFHFony) and it seems to work as intended with the given example(s). Note that the proposed 'quick-win' only supports this for Double-density single- AND double-sided images; and did not extend the format to the 184320 and 163840 sizes/formats as I'm unfamiliar with these ...

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

No branches or pull requests

2 participants