-
Notifications
You must be signed in to change notification settings - Fork 2
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
a bit smaller code #1
base: master
Are you sure you want to change the base?
Conversation
Many thanks for the code size reductions! I'll take a close look through this soon. I see that you also fixed a bug with an immediate LDA. Awesome. :) |
@peterferrie Question for you- did you happen to test these changes on a IIe with an AppleMouse card, a IIc, and a IIc+? If so, that would save me doing it. |
I didn't test on hardware, only in emulation of MAME and AppleWin. |
I just tested this in Virtual II (with an Enhanced IIe ROM image) and the vertical tracking does not appear to be working. The Y coordinate in the test application loops 0-7 twice instead of going 0-23 |
I also tested on a real IIc Plus, and it crashes upon launch of the mousedemo application. |
Bummer. I will investigate tomorrow.
|
The crash might be fixed now. |
scratch that. The Virtual II thing might be fixed by the bug fix. |
No description provided.