univrsal / input-overlay

Show keyboard, gamepad and mouse input on stream
GNU General Public License v2.0
2.68k stars 239 forks source link

Error in input_data.hpp #321

Closed patlefort closed 1 year ago

patlefort commented 1 year ago

Describe the bug Simple mistake in this line: https://github.com/univrsal/input-overlay/blob/e89fca51dbe81b1e7ab7e313084d461c4be6e122/deps/common/input_data.hpp#L97 Mixing j and i variables.

Additional information:

Additional context Compiling with warnings turned on on GCC catched this error:

input-overlay/deps/common/input_data.hpp:97:26: warning: unused variable ‘j’ [-Wunused-variable]
   97 |             for (uint8_t j = 0; i < button_count; i++) {
univrsal commented 1 year ago

Yeah I know, those functions aren't used anyways, I just forgot to remove them.

univrsal commented 1 year ago

Fixed