[coreboot-gerrit] New patch to review for coreboot: 758f8d1 libpayload: Fix prototype warnings in PDCurses

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Mon Mar 25 23:57:25 CET 2013


Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2908

-gerrit

commit 758f8d14be750caa12c85c85c0eccf98373e1b4c
Author: Stefan Reinauer <reinauer at chromium.org>
Date:   Mon Mar 25 15:38:21 2013 -0700

    libpayload: Fix prototype warnings in PDCurses
    
    This fixes the following PDCurses warnings:
    
        CC         curses/pdcurses-backend/pdcsetsc.libcurses.o
    curses/pdcurses-backend/pdcsetsc.c: In function 'PDC_curs_set':
    curses/pdcurses-backend/pdcsetsc.c:17:9: warning: implicit declaration of function 'serial_cursor_enable' [-Wimplicit-function-declaration]
    curses/pdcurses-backend/pdcsetsc.c:22:9: warning: implicit declaration of function 'video_console_cursor_enable' [-Wimplicit-function-declaration]
        CC         curses/pdcurses-backend/pdcutil.libcurses.o
    curses/pdcurses-backend/pdcutil.c:30:6: warning: no previous prototype for 'curses_enable_serial' [-Wmissing-prototypes]
    curses/pdcurses-backend/pdcutil.c:35:6: warning: no previous prototype for 'curses_enable_vga' [-Wmissing-prototypes]
    curses/pdcurses-backend/pdcutil.c:40:5: warning: function declaration isn't a prototype [-Wstrict-prototypes]
    curses/pdcurses-backend/pdcutil.c:45:5: warning: function declaration isn't a prototype [-Wstrict-prototypes]
    
    Change-Id: If0d4d475d3006f1a77f67ec46c6bdf4ee2906981
    Signed-off-by: Stefan Reinauer <reinauer at google.com>
---
 payloads/libpayload/curses/PDCurses-3.4/curses.h       | 8 ++++++++
 payloads/libpayload/curses/pdcurses-backend/pdcsetsc.c | 1 +
 payloads/libpayload/curses/pdcurses-backend/pdcutil.c  | 5 +++--
 3 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/payloads/libpayload/curses/PDCurses-3.4/curses.h b/payloads/libpayload/curses/PDCurses-3.4/curses.h
index 40585d3..5f651fe 100644
--- a/payloads/libpayload/curses/PDCurses-3.4/curses.h
+++ b/payloads/libpayload/curses/PDCurses-3.4/curses.h
@@ -1374,4 +1374,12 @@ int     sb_refresh(void);
 }
 #endif
 
+/* Local functions not defined in normal curses */
+
+void curses_enable_vga(int);
+void curses_enable_serial(int);
+
+int curses_vga_enabled(void);
+int curses_serial_enabled(void);
+
 #endif  /* __PDCURSES__ */
diff --git a/payloads/libpayload/curses/pdcurses-backend/pdcsetsc.c b/payloads/libpayload/curses/pdcurses-backend/pdcsetsc.c
index f0e83cf..d016063 100644
--- a/payloads/libpayload/curses/pdcurses-backend/pdcsetsc.c
+++ b/payloads/libpayload/curses/pdcurses-backend/pdcsetsc.c
@@ -1,6 +1,7 @@
 /* Public Domain Curses */
 /* This file is BSD licensed, Copyright 2011 secunet AG */
 
+#include <libpayload.h>
 #include "lppdc.h"
 
 int PDC_curs_set(int visibility)
diff --git a/payloads/libpayload/curses/pdcurses-backend/pdcutil.c b/payloads/libpayload/curses/pdcurses-backend/pdcutil.c
index 8fb3e16..030f462 100644
--- a/payloads/libpayload/curses/pdcurses-backend/pdcutil.c
+++ b/payloads/libpayload/curses/pdcurses-backend/pdcutil.c
@@ -2,6 +2,7 @@
 /* This file is BSD licensed, Copyright 2011 secunet AG */
 
 #include "lppdc.h"
+#include <curses.h>
 #include <libpayload.h>
 
 int curses_flags = F_ENABLE_SERIAL | F_ENABLE_CONSOLE;
@@ -37,12 +38,12 @@ void curses_enable_vga(int enable)
     curses_flags = (curses_flags & ~F_ENABLE_CONSOLE) | (enable * F_ENABLE_CONSOLE);
 }
 
-int curses_serial_enabled()
+int curses_serial_enabled(void)
 {
     return !!(curses_flags & F_ENABLE_SERIAL);
 }
 
-int curses_vga_enabled()
+int curses_vga_enabled(void)
 {
     return !!(curses_flags & F_ENABLE_CONSOLE);
 }



More information about the coreboot-gerrit mailing list