usbkeyboard: fix double printing of input

[jamesy0ung]

See issue: #19

The previous implementation caused duplicate character output due to both the keyPress_cb function and the main loop handling keyboard input. The keyPress_cb function directly printed characters, while the main loop read and reprinted them using getchar().

Now, the main loop uses non-blocking getchar() for centralized character handling, ensuring controlled output of printable and newline characters. The keyPress_cb function handles the escape key to exit the application.

Key changes:

• Removed putchar() calls from keyPress_cb in usbkeyboard.c.
  

• Updated the main loop in usbkeyboard.c to manage character printing centrally using non-blocking getchar().
  

[DacoTaco]

Hey!
thanks for the fix!

sorry i didn't reply sooner, been a bit busy.
would you mind rechecking the change and checking the space/tabs? the PR is changing way too much on first look :)

[jamesy0ung]

Try the latest commit, I clang-formatted it to tabs :)

[DacoTaco]

thanks for that! much better changelog and easier to review :)
its a very minor nitpick but i would have kept the original keyboard cb, but move the exit to the main loop.
isn't that possible/better?
shows the usage of the cb (and putchar) and the check for esc as exit

[jamesy0ung]

I have moved the exit check to the main loop.

A side effect, is that since the character print occurs before the exit check, an arrow is printed to console when esc is pressed.

[DacoTaco]

fix merged! thanks for your effort! ^^