Re: [Lost] [Patch] Kompilieren mit -Wstrict-prototypes und ensprechende Korrekturen

[Kevin Wolf <mail@xxxxxxxxxxxxx>]

Am Sonntag, 31. August 2008 17:45:04 schrieb Antoine Kaufmann:
> Index: trunk/src/modules/lib/servmgr.c
> ===================================================================
> --- trunk.orig/src/modules/lib/servmgr.c
> +++ trunk/src/modules/lib/servmgr.c
> @@ -38,8 +38,9 @@
>  #include <rpc.h>
>  #include <services.h>
>  #include <syscall.h>
> +#include <services.h>

Reicht das einmal nicht?

>
> -static pid_t servmgr_pid();
> +static pid_t servmgr_pid(void);
>
>  bool servmgr_need(const char* service_name)
>  {
> Index: trunk/src/kernel/src/console.c
> ===================================================================
> --- trunk.orig/src/kernel/src/console.c
> +++ trunk/src/kernel/src/console.c
> @@ -87,7 +87,7 @@
>  #define SCREEN_WIDTH 80
>  #define SCREEN_HEIGHT 25
>
> -void con_set_hw_cursor();
> +static void con_set_hw_cursor(void);
>
>  static unsigned short * vidmem = (unsigned short*)0xb8000;
>
> @@ -148,7 +148,7 @@ static void con_set_cursor_pos(unsigned
>  /**
>   * Leert den Bildschirm
>   */
> -static void con_clear_screen()
> +static void con_clear_screen(void)
>  {
>      int i;
>
> @@ -163,7 +163,7 @@ static void con_clear_screen()
>  /**
>   * Scrollt eine Zeile nach unten
>   */
> -static void scroll_down()
> +static void scroll_down(void)
>  {
>      int i;
>
> @@ -598,7 +598,7 @@ void init_console(void)
>  /**
>   * Aktualisiert die Postition des Hardware Cursors
>   */
> -void con_set_hw_cursor()
> +static void con_set_hw_cursor()

Hier kein void in den Klammern?

Der Rest stimmt vermutlich so halbwegs. ;-)