-
Notifications
You must be signed in to change notification settings - Fork 123
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
Improved and enhanced the floppy emulation #135
Conversation
Volker, Ben - what is the testing / level of confidence needed to merge it ? |
BTW, Ben - do you have access to 'Squash and merge' button ? |
I tested all floppy sizes with DOS and FreeDOS, most sizes with Win98 and Win2k, and all sizes above 360k with WinXP. I know the that Scan functions may or may not be correct, though I have no way of testing them other than my own code. I know of no Guest that supports these functions. Every physical hardware I tested with, even though the specifications say the Scan functions are supported with that hardware, return "illegal command". It would be good if Volker can make a few tests too (please), but I feel confident to merge.
I do not have access to either. It still says:
Thanks, |
I added a few very minor fixes and cleanups. This should be it, ready for merging. Thanks. |
This improves and enhances the floppy emulation.
This adds the ability to specify a controller type.
This adds all but one, known, controller command.
This fixes the format command. Issue #123
This fixes the error with older Windows guests.