Uzebox / uzebox

A retro-minimalist game console engine for the ATMega644
http://uzebox.org
128 stars 43 forks source link

Extra space between joypad1_status_lo and joypad1_status_lo #68

Closed bardes closed 7 years ago

bardes commented 7 years ago

I've been working on a hand optimized function to read the controllers and noticed that there seems to be an extra byte between the high and low portions of both joypad statuses. I'm not sure if it's intentional or not, but I'm opening an issue just in case...

Jubatian commented 7 years ago

That part is a little messed up, but those extra bytes are necessary and are used, they are needed by the SNES mouse (See ReadJoypadExt and ReadJoypad above it, combined they use all bytes). It could be fixed, though so that if the user doesn't need the SNES mouse, 2 bytes less would be allocated.

bardes commented 7 years ago

Oh, didn't know about that... Thanks for the quick answer!