Skip to content

a bit smaller code#1

Open
peterferrie wants to merge 10 commits into
blondie7575:masterfrom
peterferrie:master
Open

a bit smaller code#1
peterferrie wants to merge 10 commits into
blondie7575:masterfrom
peterferrie:master

Conversation

@peterferrie

Copy link
Copy Markdown

No description provided.

@blondie7575

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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

blondie7575 commented Jan 1, 2017

Copy link
Copy Markdown
Owner

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
Copy Markdown
Owner

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

@peterferrie

peterferrie commented Jan 2, 2017 via email

Copy link
Copy Markdown
Author

@peterferrie

Copy link
Copy Markdown
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
Copy Markdown
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