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

Old copperlist pointer not saved to CopperSave on startup. #8

Closed
tastymushroom opened this issue Jan 29, 2021 · 4 comments
Closed

Old copperlist pointer not saved to CopperSave on startup. #8

tastymushroom opened this issue Jan 29, 2021 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@tastymushroom
Copy link

tastymushroom commented Jan 29, 2021

Hi,
First of all fantastic project. Thanks for the effort!

I noticed a small bug in gencop.s
Line 35 you save the old copper pointer to d4, but then you don't store it in CopperSave.
So line 36 should be something like:
move.l d4,CopperSave

Then the correct value will be restored at the end.
I did try to push a fix onto a branch but no permissions to do so :-)

Cheers
TastyMushroom

@prb28
Copy link
Owner

prb28 commented Jan 30, 2021

Thanks, I'll fix that.

@prb28 prb28 self-assigned this Jan 30, 2021
@prb28 prb28 added the bug Something isn't working label Jan 30, 2021
@tastymushroom
Copy link
Author

tastymushroom commented Feb 1, 2021 via email

@prb28
Copy link
Owner

prb28 commented Feb 1, 2021

This looks like a bug.

@prb28
Copy link
Owner

prb28 commented Feb 9, 2022

Sorry for the delay on this issue. I just forgot it....
I create a new issue on the main project.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants