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

F2 to reset emulator from debugger and what to do with spinning disk drives #460

Closed
TommyGH opened this issue Aug 11, 2017 · 3 comments
Closed
Assignees
Milestone

Comments

@TommyGH
Copy link

TommyGH commented Aug 11, 2017

Note: Originally I posted this in the closed issue #450, thinking it would "re-open" that issue. Perhaps it is better to start another issue, so here we are.

Emulator: 1.26.3.0
Debugger: 2.9.0.8

Using F2 to "reset" the emulator does not reset the disk drives to inactive state.

If the intention for using F2 to "reset" the emulator is to make it more convenient than closing and restarting AppleWin, then what should happen with disk drives that are "spinning" when you press F2 while in the debugger?

Start AppleWin in AppleII+ mode
Place a bootable disk in drive 1.
F7
bpr pc=801
F7
breakpoint hits
drive 1 disk light is on
F2 to reset emulation
F5 to "swap" disks
See a dire warning message about not swapping disks while the drive is still spinning.

Note, I think I opened an issue (#250) a long time ago about the disk drive light being on when it sometimes should not be on. I'm not sure if that complicates this issue or not.

@tomcw tomcw closed this as completed in 8fce7a8 Aug 11, 2017
@tomcw
Copy link
Contributor

tomcw commented Aug 11, 2017

bpr pc=801

btw. you can just use this shorthand: bp 801

@tomcw tomcw added this to the 1.27 milestone Aug 11, 2017
@tomcw tomcw self-assigned this Aug 11, 2017
tomcw added a commit that referenced this issue Aug 12, 2017
…iately stop the drive spinning, but F2 still does (#460)
@tomcw
Copy link
Contributor

tomcw commented Aug 12, 2017

NB. Behaviour is the same for normal emulation or debugging (stepping):

@tomcw
Copy link
Contributor

tomcw commented Sep 30, 2017

FYI, new experimental release build 1.26.3.1 here, which includes this fix.

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

No branches or pull requests

2 participants