Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Saves 45 bytes in fileioc
Changes:
util_ret_null_pop_ixwas only used by one function, so I moved it inside that function so it could be reached byjrinstead ofjpld iy, 0 \ add iy, sp \ ...was changed told hl, 3 \ add hl, sp \ ...when no more than 2 arguments are to be retrieved from the stack. This saves at 1 byte + 1F when retrieving 2 24bit arguments for example.util_set_offsetwas tailed called by one function, so I moved it to that function to remove ajp.util_ret_neg_onedoesscf \ sbc hl, hl \ ret. It just so happens thatti_Deletealso contains the instructionsscf \ sbc hl, hl \ ret, so I movedutil_ret_neg_oneto the end ofti_Delete, meaning thatutil_ret_neg_onenow takes up zero bytes of space. I did something similar withutil_ret_nullas well.ex de, hlinti_Readand usinginc.s bcto clear UBC instead ofld bc, 0inutil_is_slot_open.