Re: [PATCH v3] modules: reparieren von Pong

[Kevin Wolf <kevin@xxxxxxxxxx>]

Am 22.02.2021 um 21:07 hat Nico Mayer geschrieben:
> + Mit dem Programm Pong kann man jetzt wirklich Pong spielen!
> 
> Signed-off-by: Nico Mayer <mayerNico@xxxxxxxxxx>

[Kein volles Review, aber wir brauchen eine Testmail]

Ich glaube, du solltest wie SDL noch stdin auslesen, ansonsten landen
die ganzen Eingaben am Ende auf der Shell. Ich habe beim Testen Pong
beendet und es hat sich gleich wieder gestartet, vermutlich weil Pfeil
Hoch und Enter in der Shell rausgekommen sind.

>  src/modules/c/pong/main.c | 349 ++++++++++++++++++++++++++++++++------
>  1 file changed, 294 insertions(+), 55 deletions(-)
> 
> diff --git a/src/modules/c/pong/main.c b/src/modules/c/pong/main.c
> index 0773dc21..b6205f8a 100644
> --- a/src/modules/c/pong/main.c
> +++ b/src/modules/c/pong/main.c
> @@ -33,15 +33,271 @@
>  #include <syscall.h>
>  #include <types.h>
>  #include <unistd.h>
> +#include <init.h>
> +#include <kbd.h>
> +#include <lostio.h>
>  
> -int main(int argc, char** argv)
> +#ifndef KEYCODE_ARROW_UP
> +#   define KEYCODE_ARROW_UP 103
> +#endif
> +
> +#ifndef KEYCODE_ARROW_DOWN
> +#   define KEYCODE_ARROW_DOWN 108
> +#endif
> +
> +#ifndef KEYCODE_ENTER
> +#   define KEYCODE_ENTER 28
> +#endif
> +
> +#ifndef KEYCODE_ESC
> +#   define KEYCODE_ESC 1
> +#endif

Wir wissen eigentlich ziemlich genau, was die libc an Konstanten
definiert. Drei von den Definitionen sind toter Code und bei KEYCODE_ESC
wissen wir, dass es immer benutzt wird.

Ich würde lieber KEYCODE_ESC in kbd.h hinzufügen und das hier alles
weglassen.

Kevin