Nukey Shay Posted December 26, 2019 Share Posted December 26, 2019 Changing the JMP instructions to JSR is fine if you don't mind that the stack would then be nested 2 bytes higher. Losing 3 bytes of Rom (by adding the new JSR for the crosshair) might be the better option once Ram starts getting scarce Quote Link to comment Share on other sites More sharing options...
Mallard Games Posted January 3, 2020 Author Share Posted January 3, 2020 Oops sorry, that is very strange considering the only code i change was breaking up the checkstate subroute into 2 parts. The portion that actually does the bankswitching was moved into the macro and that's pretty much it. Technically it shouldn't of have borked, but it did...? ? Quote Link to comment Share on other sites More sharing options...
Nukey Shay Posted January 6, 2020 Share Posted January 6, 2020 (edited) You're missing the actual -reason- it failed...by breaking up the routine which used to reside in one bank, it now needs to rely on the return portion of the macro between positioning the two objects...which sends it back to the address and bank of the last JSR instruction executed. JMP does not preserve return information on the stack, so the player's crosshair object was never given the chance to be set in place. Edited January 6, 2020 by Nukey Shay Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.