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

a bit smaller code #1

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

peterferrie
Copy link

No description provided.

@blondie7575
Copy link
Owner

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. :)

@blondie7575
Copy link
Owner

@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.

@peterferrie
Copy link
Author

I didn't test on hardware, only in emulation of MAME and AppleWin.
No obvious differences in behavior in either of those, but please try it for real just to be certain.

@blondie7575
Copy link
Owner

blondie7575 commented Jan 1, 2017

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

@blondie7575
Copy link
Owner

I also tested on a real IIc Plus, and it crashes upon launch of the mousedemo application.

@peterferrie
Copy link
Author

peterferrie commented Jan 2, 2017 via email

@peterferrie
Copy link
Author

The crash might be fixed now.
Separately, can you confirm that in Virtual II, the original did not have that tracking behaviour?

@peterferrie
Copy link
Author

scratch that. The Virtual II thing might be fixed by the bug fix.

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