enable Wall warnings by default

still disable it in some cases (instead of fixing) to avoid
drifting from upstream projects too much.

diff --git a/Makefile b/Makefile
index d6e968f..335a03a 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -5,7 +5,7 @@ LD = $(CROSS_COMPILE)ld
 
 ARCH = $(shell $(CC) -v 2>&1 | grep -i 'target:' | awk '{print $$2}' | awk -F '-' '{print $$1}')
 
-CFLAGS += -ggdb -Ifrontend
+CFLAGS += -Wall -ggdb -Ifrontend
 LDFLAGS += -lz -lpthread -ldl -lpng -lbz2
 ifeq "$(ARCH)" "arm"
 CFLAGS += -mcpu=cortex-a8 -mtune=cortex-a8 -mfloat-abi=softfp -ffast-math
@@ -36,6 +36,9 @@ OBJS += libpcsxcore/cdriso.o libpcsxcore/cdrom.o libpcsxcore/cheat.o libpcsxcore
 ifeq "$(ARCH)" "arm"
 OBJS += libpcsxcore/gte_neon.o
 endif
+libpcsxcore/cdrom.o libpcsxcore/misc.o: CFLAGS += -Wno-pointer-sign
+libpcsxcore/misc.o libpcsxcore/psxbios.o: CFLAGS += -Wno-nonnull
+
 # dynarec
 ifndef NO_NEW_DRC
 OBJS += libpcsxcore/new_dynarec/new_dynarec.o libpcsxcore/new_dynarec/linkage_arm.o
@@ -44,6 +47,7 @@ endif
 OBJS += libpcsxcore/new_dynarec/emu_if.o
 libpcsxcore/new_dynarec/new_dynarec.o: libpcsxcore/new_dynarec/assem_arm.c \
        libpcsxcore/new_dynarec/pcsxmem_inline.c
+libpcsxcore/new_dynarec/new_dynarec.o: CFLAGS += -Wno-all -Wno-pointer-sign
 ifdef DRC_DBG
 libpcsxcore/new_dynarec/emu_if.o: CFLAGS += -D_FILE_OFFSET_BITS=64
 CFLAGS += -DDRC_DBG
@@ -54,7 +58,6 @@ OBJS += plugins/dfsound/dma.o plugins/dfsound/freeze.o \
        plugins/dfsound/registers.o plugins/dfsound/spu.o
 plugins/dfsound/spu.o: plugins/dfsound/adsr.c plugins/dfsound/reverb.c \
        plugins/dfsound/xa.c
-plugins/dfsound/%.o: CFLAGS += -Wall
 ifeq "$(USE_OSS)" "1"
 plugins/dfsound/%.o: CFLAGS += -DUSEOSS
 OBJS += plugins/dfsound/oss.o
@@ -67,7 +70,7 @@ endif
 
 # gpu
 # note: code is not safe for strict-aliasing? (Castlevania problems)
-plugins/dfxvideo/%.o: CFLAGS += -Wall -fno-strict-aliasing
+plugins/dfxvideo/%.o: CFLAGS += -fno-strict-aliasing
 OBJS += plugins/dfxvideo/gpu.o
 plugins/dfxvideo/gpu.o: plugins/dfxvideo/fps.c plugins/dfxvideo/prim.c \
        plugins/dfxvideo/gpu.c plugins/dfxvideo/soft.c
@@ -79,11 +82,9 @@ OBJS += plugins/dfxvideo/draw_fb.o
 endif
 
 # cdrcimg
-plugins/cdrcimg/%.o: CFLAGS += -Wall
 OBJS += plugins/cdrcimg/cdrcimg.o
 
 # dfinput
-plugins/dfinput/%.o: CFLAGS += -Wall
 OBJS += plugins/dfinput/pad.o
 
 # gui
@@ -91,7 +92,6 @@ OBJS += frontend/main.o frontend/plugin.o
 ifeq "$(USE_GTK)" "1"
 OBJS += maemo/hildon.o maemo/main.o
 maemo/%.o: maemo/%.c
-maemo/%.o: CFLAGS += -Wall
 else
 OBJS += frontend/plugin_lib.o frontend/menu.o
 OBJS += frontend/linux/fbdev.o frontend/linux/in_evdev.o
@@ -114,7 +114,7 @@ endif
 ifdef PCNT
 CFLAGS += -DPCNT
 endif
-frontend/%.o: CFLAGS += -Wall -DIN_EVDEV
+frontend/%.o: CFLAGS += -DIN_EVDEV
 frontend/menu.o: frontend/revision.h
 
 frontend/revision.h: FORCE

diff --git a/libpcsxcore/cdriso.c b/libpcsxcore/cdriso.c
index 2d371dd..eb92281 100644 (file)
--- a/libpcsxcore/cdriso.c
+++ b/libpcsxcore/cdriso.c
@@ -833,7 +833,7 @@ static long CALLBACK ISOgetTD(unsigned char track, unsigned char *buffer) {
                unsigned int sect;
                unsigned char time[3];
                sect = msf2sec(ti[numtracks].start) + msf2sec(ti[numtracks].length);
-               sec2msf(sect, time);
+               sec2msf(sect, (char *)time);
                buffer[2] = time[0];
                buffer[1] = time[1];
                buffer[0] = time[2];
@@ -913,7 +913,7 @@ static long CALLBACK ISOplay(unsigned char *time) {
                return 0;
 
        // find the track
-       sect = msf2sec(time);
+       sect = msf2sec((char *)time);
        for (i = numtracks; i > 1; i--)
                if (msf2sec(ti[i].start) <= sect + 2 * 75)
                        break;

diff --git a/libpcsxcore/ppf.c b/libpcsxcore/ppf.c
index 268ed1c..ee32d8b 100644 (file)
--- a/libpcsxcore/ppf.c
+++ b/libpcsxcore/ppf.c
@@ -335,7 +335,7 @@ void BuildPPFCache() {
 unsigned char *sbi_sectors;
 
 int LoadSBI(const char *fname, int sector_count) {
-       char buffer[16], sbifile[MAXPATHLEN];
+       char buffer[16];
        FILE *sbihandle;
        u8 sbitime[3];
        int s;
@@ -344,7 +344,6 @@ int LoadSBI(const char *fname, int sector_count) {
        if (sbihandle == NULL)
                return -1;
 
-if (sbi_sectors != NULL) printf("sbi_sectors?\n");
        sbi_sectors = calloc(1, sector_count / 8);
        if (sbi_sectors == NULL)
                return -1;

diff --git a/libpcsxcore/ppf.h b/libpcsxcore/ppf.h
index fdb11eb..aad85c9 100644 (file)
--- a/libpcsxcore/ppf.h
+++ b/libpcsxcore/ppf.h
@@ -32,6 +32,8 @@ void UnloadSBI(void);
 
 extern unsigned char *sbi_sectors;
 
+#include "cdrom.h"
+
 static inline int CheckSBI(const u8 *t)
 {
        int s;

diff --git a/libpcsxcore/psxcommon.c b/libpcsxcore/psxcommon.c
index ea655df..8313304 100644 (file)
--- a/libpcsxcore/psxcommon.c
+++ b/libpcsxcore/psxcommon.c
@@ -57,7 +57,12 @@ void EmuUpdate() {
                SysUpdate();
 
        ApplyCheats();
-       pl_frame_limit();
+
+       // reamed hack
+       {
+               extern void pl_frame_limit(void);
+               pl_frame_limit();
+       }
 }
 
 void __Log(char *fmt, ...) {

diff --git a/libpcsxcore/psxcounters.c b/libpcsxcore/psxcounters.c
index 1c514e8..f6c1bc4 100644 (file)
--- a/libpcsxcore/psxcounters.c
+++ b/libpcsxcore/psxcounters.c
@@ -100,7 +100,7 @@ void verboseLog( s32 level, const char *str, ... )
         vsprintf( buf, str, va );
         va_end( va );
 
-        printf( buf );
+        printf( "%s", buf );
         fflush( stdout );
     }
 }

diff --git a/libpcsxcore/sio.c b/libpcsxcore/sio.c
index 8a30797..df130e8 100644 (file)
--- a/libpcsxcore/sio.c
+++ b/libpcsxcore/sio.c
@@ -673,7 +673,7 @@ void ConvertMcd(char *mcd, char *data) {
 }
 
 void GetMcdBlockInfo(int mcd, int block, McdBlock *Info) {
-       unsigned char *data = NULL, *ptr, *str, *sstr;
+       char *data = NULL, *ptr, *str, *sstr;
        unsigned short clut[16];
        unsigned short c;
        int i, x;

diff --git a/libpcsxcore/socket.c b/libpcsxcore/socket.c
index 2e0dc87..31f82e2 100644 (file)
--- a/libpcsxcore/socket.c
+++ b/libpcsxcore/socket.c
@@ -190,7 +190,7 @@ int ReadSocket(char * buffer, int len) {
 }
 
 int RawReadSocket(char * buffer, int len) {
-    int r;
+    int r = 0;
     int mlen = len < ptr ? len : ptr;
 
     if (!client_socket)