Fixed some warnings when using -Weffc++ gcc option

[stockfish] / src / Makefile

diff --git a/src/Makefile b/src/Makefile
index 7459d06d3f0346ca80da60d8d04e5c81002c1d33..b08b15f3272a60fe3168c7d6d121775f0e2dfd7d 100644 (file)
--- a/src/Makefile
+++ b/src/Makefile
@@ -35,8 +35,8 @@ PGOBENCH = ./$(EXE) bench 32 1 10 default depth
 ### Object files
 OBJS = application.o bitboard.o pawns.o material.o endgame.o evaluate.o main.o \
        misc.o move.o movegen.o history.o movepick.o search.o piece.o \
-       position.o direction.o tt.o value.o uci.o ucioption.o \
-       mersenne.o book.o bitbase.o san.o benchmark.o
+       position.o direction.o tt.o uci.o ucioption.o \
+       mersenne.o book.o bitbase.o san.o benchmark.o timeman.o
 
 
 ### ==========================================================================
@@ -219,10 +219,14 @@ ifeq ($(COMP),icc)
 endif
 
 ### 3.2 General compiler settings
-CXXFLAGS += -g -Wall -fno-exceptions -fno-rtti $(EXTRACXXFLAGS)
+CXXFLAGS = -g -Wall -Wcast-qual -fno-exceptions -fno-rtti $(EXTRACXXFLAGS)
+
+ifeq ($(comp),gcc)
+       CXXFLAGS += -ansi -pedantic -Wno-long-long -Wextra
+endif
 
 ifeq ($(comp),icc)
-       CXXFLAGS += -wd383,869,981,10187,10188,11505,11503
+       CXXFLAGS += -wd383,981,10187,10188,11505,11503 -Wcheck -Wabi -Wdeprecated -strict-ansi
 endif
 
 ifeq ($(os),osx)
@@ -230,7 +234,7 @@ ifeq ($(os),osx)
 endif
 
 ### 3.3 General linker settings
-LDFLAGS += -lpthread $(EXTRALDFLAGS)
+LDFLAGS = -lpthread $(EXTRALDFLAGS)
 
 ifeq ($(os),osx)
        LDFLAGS += -arch $(arch)
@@ -278,8 +282,10 @@ endif
 
 ### 3.8 prefetch
 ifeq ($(prefetch),yes)
-       CXXFLAGS += -msse -DUSE_PREFETCH
+       CXXFLAGS += -msse
        DEPENDFLAGS += -msse
+else
+       CXXFLAGS += -DNO_PREFETCH
 endif
 
 ### 3.9 bsfq
@@ -406,7 +412,7 @@ install:
        -strip $(BINDIR)/$(EXE)
 
 clean:
-       $(RM) $(EXE) *.o .depend *~ core bench.txt
+       $(RM) $(EXE) *.o .depend *~ core bench.txt *.gcda
 
 testrun:
        @$(PGOBENCH)
@@ -498,7 +504,7 @@ icc-profile-clean:
 
 hpux:
        $(MAKE) \
-       CXX='/opt/aCC/bin/aCC -AA +hpxstd98 -DBIGENDIAN -mt +O3 -DNDEBUG' \
+       CXX='/opt/aCC/bin/aCC -AA +hpxstd98 -DBIGENDIAN -mt +O3 -DNDEBUG -DNO_PREFETCH' \
        CXXFLAGS="" \
        LDFLAGS="" \
        all