Hi Kevin,
Thanks for taking the time to review the patch.
You are right that a hardcoded 4 is perfect for the 16-bit Z80
(covering two pushed register pairs), but it would fall short for the eZ80 ADL
mode where two pushed registers take up six bytes.
Making the limit dependent on the architecture's address width is a reasonable
solution. Since `addr_len` is already available in the function, I will change
it to `2 * addr_len`. What do you think?
I will update the comments as you suggested:
1. Mentioned conditional CALLs alongside the unconditional ones.
2. Updated the old "overflow" comment inside the break block to accurately
reflect both exit conditions.
Best regards,
Ronald