define common build flags so they can easily be changed for all tools
Modified: trunk/reactos/tools/bin2c.mak
Modified: trunk/reactos/tools/bin2res/bin2res.mak
Modified: trunk/reactos/tools/buildno/buildno.mak
Modified: trunk/reactos/tools/cabman/cabman.mak
Modified: trunk/reactos/tools/cdmake/cdmake.c
Modified: trunk/reactos/tools/cdmake/cdmake.mak
Modified: trunk/reactos/tools/gendib/gendib.mak
Modified: trunk/reactos/tools/mkhive/mkhive.mak
Modified: trunk/reactos/tools/nci/nci.mak
Modified: trunk/reactos/tools/rbuild/rbuild.mak
Modified: trunk/reactos/tools/rsym.mak
Modified: trunk/reactos/tools/tools.mak
Modified: trunk/reactos/tools/unicode/unicode.mak
Modified: trunk/reactos/tools/widl/widl.mak
Modified: trunk/reactos/tools/winebuild/winebuild.mak
Modified: trunk/reactos/tools/wmc/wmc.mak
Modified: trunk/reactos/tools/wpp/wpp.mak
Modified: trunk/reactos/tools/wrc/wrc.mak

Modified: trunk/reactos/tools/bin2c.mak
--- trunk/reactos/tools/bin2c.mak	2005-08-01 10:21:20 UTC (rev 16940)
+++ trunk/reactos/tools/bin2c.mak	2005-08-01 10:58:28 UTC (rev 16941)
@@ -15,9 +15,9 @@
 BIN2C_OBJECTS = \
 	$(addprefix $(INTERMEDIATE_), $(BIN2C_SOURCES:.c=.o))
 
-BIN2C_HOST_CFLAGS = -g -Werror -Wall
+BIN2C_HOST_CFLAGS = $(TOOLS_CFLAGS)
 
-BIN2C_HOST_LFLAGS = -g
+BIN2C_HOST_LFLAGS = $(TOOLS_LFLAGS)
 
 .PHONY: bin2c
 bin2c: $(BIN2C_TARGET)

Modified: trunk/reactos/tools/bin2res/bin2res.mak
--- trunk/reactos/tools/bin2res/bin2res.mak	2005-08-01 10:21:20 UTC (rev 16940)
+++ trunk/reactos/tools/bin2res/bin2res.mak	2005-08-01 10:58:28 UTC (rev 16941)
@@ -26,9 +26,9 @@
 BIN2RES_OBJECTS = \
 	$(addprefix $(INTERMEDIATE_), $(BIN2RES_SOURCES:.c=.o))
 
-BIN2RES_HOST_CFLAGS = -Iinclude/wine -D__REACTOS__ -g -Werror -Wall
+BIN2RES_HOST_CFLAGS = -Iinclude/wine -D__REACTOS__ $(TOOLS_CFLAGS)
 
-BIN2RES_HOST_LFLAGS = -g
+BIN2RES_HOST_LFLAGS = $(TOOLS_LFLAGS)
 
 $(BIN2RES_TARGET): $(BIN2RES_OBJECTS) | $(BIN2RES_OUT)
 	$(ECHO_LD)

Modified: trunk/reactos/tools/buildno/buildno.mak
--- trunk/reactos/tools/buildno/buildno.mak	2005-08-01 10:21:20 UTC (rev 16940)
+++ trunk/reactos/tools/buildno/buildno.mak	2005-08-01 10:58:28 UTC (rev 16941)
@@ -28,9 +28,9 @@
 BUILDNO_OBJECTS = \
   $(addprefix $(INTERMEDIATE_), $(BUILDNO_SOURCES:.cpp=.o))
 
-BUILDNO_HOST_CFLAGS = -Iinclude/reactos -g -Werror -Wall
+BUILDNO_HOST_CXXFLAGS = -Iinclude/reactos $(TOOLS_CPPFLAGS)
 
-BUILDNO_HOST_LFLAGS = -g
+BUILDNO_HOST_LFLAGS = $(TOOLS_LFLAGS)
 
 $(BUILDNO_TARGET): $(BUILDNO_OBJECTS) | $(BUILDNO_OUT)
 	$(ECHO_LD)
@@ -38,19 +38,19 @@
 
 $(BUILDNO_INT_)buildno.o: $(BUILDNO_BASE_)buildno.cpp | $(BUILDNO_INT)
 	$(ECHO_CC)
-	${host_gpp} $(BUILDNO_HOST_CFLAGS) -c $< -o $@
+	${host_gpp} $(BUILDNO_HOST_CXXFLAGS) -c $< -o $@
 
 $(BUILDNO_INT_)exception.o: $(BUILDNO_BASE_)exception.cpp | $(BUILDNO_INT)
 	$(ECHO_CC)
-	${host_gpp} $(BUILDNO_HOST_CFLAGS) -c $< -o $@
+	${host_gpp} $(BUILDNO_HOST_CXXFLAGS) -c $< -o $@
 
 $(BUILDNO_INT_)ssprintf.o: $(BUILDNO_BASE_)ssprintf.cpp | $(BUILDNO_INT)
 	$(ECHO_CC)
-	${host_gpp} $(BUILDNO_HOST_CFLAGS) -c $< -o $@
+	${host_gpp} $(BUILDNO_HOST_CXXFLAGS) -c $< -o $@
 
 $(BUILDNO_INT_)XML.o: $(BUILDNO_BASE_)XML.cpp | $(BUILDNO_INT)
 	$(ECHO_CC)
-	${host_gpp} $(BUILDNO_HOST_CFLAGS) -c $< -o $@
+	${host_gpp} $(BUILDNO_HOST_CXXFLAGS) -c $< -o $@
 
 .PHONY: buildno_clean
 buildno_clean:

Modified: trunk/reactos/tools/cabman/cabman.mak
--- trunk/reactos/tools/cabman/cabman.mak	2005-08-01 10:21:20 UTC (rev 16940)
+++ trunk/reactos/tools/cabman/cabman.mak	2005-08-01 10:58:28 UTC (rev 16941)
@@ -29,11 +29,11 @@
 CABMAN_OBJECTS = \
   $(addprefix $(INTERMEDIATE_), $(CABMAN_SOURCES:.cxx=.o))
 
-CABMAN_HOST_CFLAGS = -Iinclude/reactos -Ilib/zlib -g -Werror -Wall
+CABMAN_HOST_CXXFLAGS = -Iinclude/reactos -Ilib/zlib $(TOOLS_CXXFLAGS)
 
 CABMAN_HOST_LIBS = $(ZLIB_HOST_TARGET)
 
-CABMAN_HOST_LFLAGS = -g $(CABMAN_HOST_LIBS)
+CABMAN_HOST_LFLAGS = $(TOOLS_LFLAGS) $(CABMAN_HOST_LIBS)
 
 .PHONY: cabman
 cabman: $(CABMAN_TARGET)
@@ -44,23 +44,23 @@
 
 $(CABMAN_INT_)cabinet.o: $(CABMAN_BASE_)cabinet.cxx | $(CABMAN_INT)
 	$(ECHO_CC)
-	${host_gpp} $(CABMAN_HOST_CFLAGS) -c $< -o $@
+	${host_gpp} $(CABMAN_HOST_CXXFLAGS) -c $< -o $@
 
 $(CABMAN_INT_)dfp.o: $(CABMAN_BASE_)dfp.cxx | $(CABMAN_INT)
 	$(ECHO_CC)
-	${host_gpp} $(CABMAN_HOST_CFLAGS) -c $< -o $@
+	${host_gpp} $(CABMAN_HOST_CXXFLAGS) -c $< -o $@
 
 $(CABMAN_INT_)main.o: $(CABMAN_BASE_)main.cxx | $(CABMAN_INT)
 	$(ECHO_CC)
-	${host_gpp} $(CABMAN_HOST_CFLAGS) -c $< -o $@
+	${host_gpp} $(CABMAN_HOST_CXXFLAGS) -c $< -o $@
 
 $(CABMAN_INT_)mszip.o: $(CABMAN_BASE_)mszip.cxx | $(CABMAN_INT)
 	$(ECHO_CC)
-	${host_gpp} $(CABMAN_HOST_CFLAGS) -c $< -o $@
+	${host_gpp} $(CABMAN_HOST_CXXFLAGS) -c $< -o $@
 
 $(CABMAN_INT_)raw.o: $(CABMAN_BASE_)raw.cxx | $(CABMAN_INT)
 	$(ECHO_CC)
-	${host_gpp} $(CABMAN_HOST_CFLAGS) -c $< -o $@
+	${host_gpp} $(CABMAN_HOST_CXXFLAGS) -c $< -o $@
 
 .PHONY: cabman_clean
 cabman_clean:

Modified: trunk/reactos/tools/cdmake/cdmake.c
--- trunk/reactos/tools/cdmake/cdmake.c	2005-08-01 10:21:20 UTC (rev 16940)
+++ trunk/reactos/tools/cdmake/cdmake.c	2005-08-01 10:58:28 UTC (rev 16941)
@@ -423,6 +423,9 @@
         if (d->extension_on_cd[0] != 0)
           identifier_size += 1 + strlen(d->extension_on_cd);
         break;
+      default:
+        identifier_size = 1;
+        break;
     }
   }
   record_size = 33 + identifier_size;

Modified: trunk/reactos/tools/cdmake/cdmake.mak
--- trunk/reactos/tools/cdmake/cdmake.mak	2005-08-01 10:21:20 UTC (rev 16940)
+++ trunk/reactos/tools/cdmake/cdmake.mak	2005-08-01 10:58:28 UTC (rev 16941)
@@ -26,9 +26,9 @@
 CDMAKE_OBJECTS = \
 	$(addprefix $(INTERMEDIATE_), $(CDMAKE_SOURCES:.c=.o))
 
-CDMAKE_HOST_CFLAGS = -Iinclude -g -Werror -Wall
+CDMAKE_HOST_CFLAGS = -Iinclude $(TOOLS_CFLAGS)
 
-CDMAKE_HOST_LFLAGS = -g
+CDMAKE_HOST_LFLAGS = $(TOOLS_LFLAGS)
 
 .PHONY: cdmake
 cdmake: $(CDMAKE_TARGET)

Modified: trunk/reactos/tools/gendib/gendib.mak
--- trunk/reactos/tools/gendib/gendib.mak	2005-08-01 10:21:20 UTC (rev 16940)
+++ trunk/reactos/tools/gendib/gendib.mak	2005-08-01 10:58:28 UTC (rev 16941)
@@ -24,9 +24,9 @@
 GENDIB_OBJECTS = \
     $(addprefix $(INTERMEDIATE_), $(GENDIB_SOURCES:.c=.o))
 
-GENDIB_HOST_CFLAGS = -g -Werror -Wall
+GENDIB_HOST_CFLAGS = $(TOOLS_CFLAGS)
 
-GENDIB_HOST_LFLAGS = -g
+GENDIB_HOST_LFLAGS = $(TOOLS_LFLAGS)
 
 $(GENDIB_TARGET): $(GENDIB_OBJECTS) | $(GENDIB_OUT)
 	$(ECHO_LD)

Modified: trunk/reactos/tools/mkhive/mkhive.mak
--- trunk/reactos/tools/mkhive/mkhive.mak	2005-08-01 10:21:20 UTC (rev 16940)
+++ trunk/reactos/tools/mkhive/mkhive.mak	2005-08-01 10:58:28 UTC (rev 16941)
@@ -29,9 +29,9 @@
 MKHIVE_OBJECTS = \
 	$(addprefix $(INTERMEDIATE_), $(MKHIVE_SOURCES:.c=.o))
 
-MKHIVE_HOST_CFLAGS = -g -Werror -Wall
+MKHIVE_HOST_CFLAGS = $(TOOLS_CFLAGS)
 
-MKHIVE_HOST_LFLAGS = -g
+MKHIVE_HOST_LFLAGS = $(TOOLS_LFLAGS)
 
 .PHONY: mkhive
 mkhive: $(MKHIVE_TARGET)

Modified: trunk/reactos/tools/nci/nci.mak
--- trunk/reactos/tools/nci/nci.mak	2005-08-01 10:21:20 UTC (rev 16940)
+++ trunk/reactos/tools/nci/nci.mak	2005-08-01 10:58:28 UTC (rev 16941)
@@ -24,9 +24,9 @@
 NCI_OBJECTS = \
     $(addprefix $(INTERMEDIATE_), $(NCI_SOURCES:.c=.o))
 
-NCI_HOST_CFLAGS = -Iinclude -g -Werror -Wall
+NCI_HOST_CFLAGS = -Iinclude $(TOOLS_CFLAGS)
 
-NCI_HOST_LFLAGS = -g
+NCI_HOST_LFLAGS = $(TOOLS_LFLAGS)
 
 $(NCI_TARGET): $(NCI_OBJECTS) | $(NCI_OUT)
 	$(ECHO_LD)

Modified: trunk/reactos/tools/rbuild/rbuild.mak
--- trunk/reactos/tools/rbuild/rbuild.mak	2005-08-01 10:21:20 UTC (rev 16940)
+++ trunk/reactos/tools/rbuild/rbuild.mak	2005-08-01 10:58:28 UTC (rev 16941)
@@ -215,9 +215,9 @@
 	$(RBUILD_COMMON_OBJECTS) \
 	$(RBUILD_TEST_SPECIAL_OBJECTS)
 
-RBUILD_HOST_CXXFLAGS = -g -I$(RBUILD_BASE) -Werror -Wall
+RBUILD_HOST_CXXFLAGS = -I$(RBUILD_BASE) $(TOOLS_CPPFLAGS)
 
-RBUILD_HOST_LFLAGS = -g
+RBUILD_HOST_LFLAGS = $(TOOLS_LFLAGS)
 
 .PHONY: rbuild
 rbuild: $(RBUILD_TARGET)

Modified: trunk/reactos/tools/rsym.mak
--- trunk/reactos/tools/rsym.mak	2005-08-01 10:21:20 UTC (rev 16940)
+++ trunk/reactos/tools/rsym.mak	2005-08-01 10:58:28 UTC (rev 16941)
@@ -16,9 +16,9 @@
 RSYM_OBJECTS = \
 	$(addprefix $(INTERMEDIATE_), $(RSYM_SOURCES:.c=.o))
 
-RSYM_HOST_CFLAGS = -g -Werror -Wall
+RSYM_HOST_CFLAGS = $(TOOLS_CFLAGS)
 
-RSYM_HOST_LFLAGS = -g
+RSYM_HOST_LFLAGS = $(TOOLS_LFLAGS)
 
 .PHONY: rsym
 rsym: $(RSYM_TARGET)

Modified: trunk/reactos/tools/tools.mak
--- trunk/reactos/tools/tools.mak	2005-08-01 10:21:20 UTC (rev 16940)
+++ trunk/reactos/tools/tools.mak	2005-08-01 10:58:28 UTC (rev 16941)
@@ -5,6 +5,10 @@
 TOOLS_OUT = $(OUTPUT_)$(TOOLS_BASE)
 TOOLS_OUT_ = $(TOOLS_OUT)$(SEP)
 
+TOOLS_CFLAGS = -Werror -Wall -Wpointer-arith -Wno-strict-aliasing
+TOOLS_CPPFLAGS = -Werror -Wall -Wpointer-arith
+TOOLS_LFLAGS =
+
 $(TOOLS_INT): | $(INTERMEDIATE)
 	$(ECHO_MKDIR)
 	${mkdir} $@

Modified: trunk/reactos/tools/unicode/unicode.mak
--- trunk/reactos/tools/unicode/unicode.mak	2005-08-01 10:21:20 UTC (rev 16940)
+++ trunk/reactos/tools/unicode/unicode.mak	2005-08-01 10:58:28 UTC (rev 16941)
@@ -97,7 +97,8 @@
 UNICODE_HOST_CFLAGS = \
 	-D__USE_W32API -DWINVER=0x501 -DWINE_UNICODE_API= \
 	-Dwchar_t="unsigned short" -D_WCHAR_T_DEFINED \
-	-I$(UNICODE_BASE) -Iinclude/wine -Iw32api/include
+	-I$(UNICODE_BASE) -Iinclude/wine -Iw32api/include \
+	$(TOOLS_CFLAGS)
 
 .PHONY: unicode
 unicode: $(UNICODE_TARGET)

Modified: trunk/reactos/tools/widl/widl.mak
--- trunk/reactos/tools/widl/widl.mak	2005-08-01 10:21:20 UTC (rev 16940)
+++ trunk/reactos/tools/widl/widl.mak	2005-08-01 10:58:28 UTC (rev 16941)
@@ -53,12 +53,12 @@
 WIDL_OBJECTS = \
   $(addprefix $(INTERMEDIATE_), $(WIDL_SOURCES:.c=.o))
 
-WIDL_HOST_CFLAGS = -g -Werror -Wall \
+WIDL_HOST_CFLAGS = $(TOOLS_CFLAGS) \
 	-DINT16=SHORT -D__USE_W32API -DYYDEBUG=1 \
 	-I$(WIDL_BASE) -I$(WPP_BASE) \
 	-Iinclude/wine -Iinclude -Iw32api/include
 
-WIDL_HOST_LFLAGS = -g
+WIDL_HOST_LFLAGS = $(TOOLS_LFLAGS)
 
 WIDL_LIBS = $(WPP_TARGET)
 

Modified: trunk/reactos/tools/winebuild/winebuild.mak
--- trunk/reactos/tools/winebuild/winebuild.mak	2005-08-01 10:21:20 UTC (rev 16940)
+++ trunk/reactos/tools/winebuild/winebuild.mak	2005-08-01 10:58:28 UTC (rev 16941)
@@ -32,9 +32,9 @@
 WINEBUILD_OBJECTS = \
   $(addprefix $(INTERMEDIATE_), $(WINEBUILD_SOURCES:.c=.o))
 
-WINEBUILD_HOST_CFLAGS = -D__USE_W32API -Iinclude/wine
+WINEBUILD_HOST_CFLAGS = $(TOOLS_CFLAGS) -D__USE_W32API -Iinclude -Iinclude/wine
 
-WINEBUILD_HOST_LFLAGS = -g
+WINEBUILD_HOST_LFLAGS = $(TOOLS_LFLAGS)
 
 .PHONY: winebuild
 winebuild: $(WINEBUILD_TARGET)

Modified: trunk/reactos/tools/wmc/wmc.mak
--- trunk/reactos/tools/wmc/wmc.mak	2005-08-01 10:21:20 UTC (rev 16940)
+++ trunk/reactos/tools/wmc/wmc.mak	2005-08-01 10:58:28 UTC (rev 16941)
@@ -32,9 +32,9 @@
 WMC_OBJECTS = \
   $(addprefix $(INTERMEDIATE_), $(WMC_SOURCES:.c=.o))
 
-WMC_HOST_CXXFLAGS = -I$(WMC_BASE) -g -Werror -Wall
+WMC_HOST_CFLAGS = -I$(WMC_BASE) $(TOOLS_CFLAGS)
 
-WMC_HOST_LFLAGS = -g
+WMC_HOST_LFLAGS = $(TOOLS_LFLAGS)
 
 .PHONY: wmc
 wmc: $(WMC_TARGET)
@@ -45,35 +45,35 @@
 
 $(WMC_INT_)getopt.o: $(WMC_BASE_)getopt.c | $(WMC_INT)
 	$(ECHO_CC)
-	${host_gcc} $(WMC_HOST_CXXFLAGS) -c $< -o $@
+	${host_gcc} $(WMC_HOST_CFLAGS) -c $< -o $@
 
 $(WMC_INT_)lang.o: $(WMC_BASE_)lang.c | $(WMC_INT)
 	$(ECHO_CC)
-	${host_gcc} $(WMC_HOST_CXXFLAGS) -c $< -o $@
+	${host_gcc} $(WMC_HOST_CFLAGS) -c $< -o $@
 
 $(WMC_INT_)mcl.o: $(WMC_BASE_)mcl.c | $(WMC_INT)
 	$(ECHO_CC)
-	${host_gcc} $(WMC_HOST_CXXFLAGS) -c $< -o $@
+	${host_gcc} $(WMC_HOST_CFLAGS) -c $< -o $@
 
 $(WMC_INT_)utils.o: $(WMC_BASE_)utils.c | $(WMC_INT)
 	$(ECHO_CC)
-	${host_gcc} $(WMC_HOST_CXXFLAGS) -c $< -o $@
+	${host_gcc} $(WMC_HOST_CFLAGS) -c $< -o $@
 
 $(WMC_INT_)wmc.o: $(WMC_BASE_)wmc.c | $(WMC_INT)
 	$(ECHO_CC)
-	${host_gcc} $(WMC_HOST_CXXFLAGS) -c $< -o $@
+	${host_gcc} $(WMC_HOST_CFLAGS) -c $< -o $@
 
 $(WMC_INT_)write.o: $(WMC_BASE_)write.c | $(WMC_INT)
 	$(ECHO_CC)
-	${host_gcc} $(WMC_HOST_CXXFLAGS) -c $< -o $@
+	${host_gcc} $(WMC_HOST_CFLAGS) -c $< -o $@
 
 $(WMC_INT_)y_tab.o: $(WMC_BASE_)y_tab.c | $(WMC_INT)
 	$(ECHO_CC)
-	${host_gcc} $(WMC_HOST_CXXFLAGS) -c $< -o $@
+	${host_gcc} $(WMC_HOST_CFLAGS) -c $< -o $@
 
 $(WMC_INT_)misc.o: $(WMC_BASE_)misc.c | $(WMC_INT)
 	$(ECHO_CC)
-	${host_gcc} $(WMC_HOST_CXXFLAGS) -c $< -o $@
+	${host_gcc} $(WMC_HOST_CFLAGS) -c $< -o $@
 
 .PHONY: wmc_clean
 wmc_clean:

Modified: trunk/reactos/tools/wpp/wpp.mak
--- trunk/reactos/tools/wpp/wpp.mak	2005-08-01 10:21:20 UTC (rev 16940)
+++ trunk/reactos/tools/wpp/wpp.mak	2005-08-01 10:58:28 UTC (rev 16941)
@@ -28,7 +28,7 @@
 WPP_OBJECTS = \
     $(addprefix $(INTERMEDIATE_), $(WPP_SOURCES:.c=.o))
 
-WPP_HOST_CFLAGS = -D__USE_W32API -I$(WPP_BASE) -Iinclude -Iinclude/wine -g
+WPP_HOST_CFLAGS = -D__USE_W32API -I$(WPP_BASE) -Iinclude -Iinclude/wine $(TOOLS_CFLAGS)
 
 .PHONY: wpp
 wpp: $(WPP_TARGET)

Modified: trunk/reactos/tools/wrc/wrc.mak
--- trunk/reactos/tools/wrc/wrc.mak	2005-08-01 10:21:20 UTC (rev 16940)
+++ trunk/reactos/tools/wrc/wrc.mak	2005-08-01 10:58:28 UTC (rev 16941)
@@ -52,13 +52,13 @@
 WRC_OBJECTS = \
   $(addprefix $(INTERMEDIATE_), $(WRC_SOURCES:.c=.o))
 
-WRC_HOST_CFLAGS = -I$(WRC_BASE) -g -Werror -Wall \
+WRC_HOST_CFLAGS = -I$(WRC_BASE) $(TOOLS_CFLAGS) \
                   -D__USE_W32API -DWINE_UNICODE_API= \
                   -Dwchar_t="unsigned short" -D_WCHAR_T_DEFINED \
                   -I$(UNICODE_BASE) -I$(WPP_BASE) \
                   -Iinclude/wine -Iinclude -Iw32api/include
 
-WRC_HOST_LFLAGS = -g
+WRC_HOST_LFLAGS = $(TOOLS_LFLAGS)
 
 WRC_LIBS = $(UNICODE_TARGET) $(WPP_TARGET)