From 19a792d17cbc40367eeff3cf7d0b61f93470774e Mon Sep 17 00:00:00 2001
From: gevaerts <gevaerts@a1c6a512-1295-4272-9138-f99709370657>
Date: Tue, 20 Oct 2009 17:07:06 +0000
Subject: [PATCH] compile checkwps with -Wall, to avoid accidentally breaking
 it again when a function prototype changes

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@23291 a1c6a512-1295-4272-9138-f99709370657
---
 tools/checkwps/checkwps.c    | 3 ++-
 tools/checkwps/checkwps.make | 2 +-
 tools/checkwps/file.h        | 9 +++++++++
 3 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/tools/checkwps/checkwps.c b/tools/checkwps/checkwps.c
index d72938b6c..4736f31e7 100644
--- a/tools/checkwps/checkwps.c
+++ b/tools/checkwps/checkwps.c
@@ -26,9 +26,11 @@
 #include "checkwps.h"
 #include "resize.h"
 #include "wps.h"
+#include "skin_engine.h"
 #include "wps_internals.h"
 #include "settings.h"
 #include "viewport.h"
+#include "file.h"
 
 bool debug_wps = true;
 int wps_verbose_level = 0;
@@ -313,7 +315,6 @@ int main(int argc, char **argv)
     int filearg = 1;
 
     struct wps_data wps;
-    struct screen* wps_screen = &screens[SCREEN_MAIN];
 
     /* No arguments -> print the help text
      * Also print the help text upon -h or --help */
diff --git a/tools/checkwps/checkwps.make b/tools/checkwps/checkwps.make
index b28d8e841..13fef935d 100644
--- a/tools/checkwps/checkwps.make
+++ b/tools/checkwps/checkwps.make
@@ -7,7 +7,7 @@
 # $Id$
 #
 
-FLAGS=-g -D__PCTOOL__ -DDEBUG -DROCKBOX_DIR_LEN=9 -DWPS_DIR=\".\" $(TARGET)
+FLAGS=-g -D__PCTOOL__ -DDEBUG -DROCKBOX_DIR_LEN=9 -DWPS_DIR=\".\" $(TARGET) -Wall
 
 SRC= $(call preprocess, $(TOOLSDIR)/checkwps/SOURCES)
 
diff --git a/tools/checkwps/file.h b/tools/checkwps/file.h
index 4256c1ee3..a14d7f5b9 100644
--- a/tools/checkwps/file.h
+++ b/tools/checkwps/file.h
@@ -5,3 +5,12 @@
 /* Wrapper - required for O_RDONLY */
 
 #include <fcntl.h>
+
+extern ssize_t read(int fd, void *buf, size_t count);
+extern ssize_t write(int fd, const void *buf, size_t count);
+extern off_t lseek(int fildes, off_t offset, int whence);
+extern int close(int fd);
+
+/* strlcpy doesn't belong here (it's in string.h in the rockbox sources),
+ * but this avoids complicated magic to override the system string.h */
+size_t  strlcpy(char *dst, const char *src, size_t siz);
-- 
2.11.4.GIT