[Xen-changelog] The attached patch adds -Werror to HOSTCFLAGS in Config.mk, makes

[Xen patchbot -unstable <patchbot-unstable@xxxxxxxxxxxxxxxxxxx>]

# HG changeset patch
# User kaf24@xxxxxxxxxxxxxxxxxxxx
# Node ID 41ceeb6828b57b082735518431cc67bce7e8916d
# Parent  c4512592a1dc11c76b94d87f14849bcc38871f72
The attached patch adds -Werror to HOSTCFLAGS in Config.mk, makes
xen/tools actually use HOSTCFLAGS (it was already using HOSTCC), and
fixes some gcc-4.0 signedness warnings in xen/tools/symbols.c.

Signed-off-by: Josh Triplett <josht@xxxxxxxxxx>

diff -r c4512592a1dc -r 41ceeb6828b5 Config.mk
--- a/Config.mk Mon Aug  8 08:18:38 2005
+++ b/Config.mk Mon Aug  8 08:32:52 2005
@@ -7,7 +7,7 @@
 
 # Tools to run on system hosting the build
 HOSTCC     = gcc
-HOSTCFLAGS = -Wall -Wstrict-prototypes -O2 -fomit-frame-pointer 
+HOSTCFLAGS = -Wall -Werror -Wstrict-prototypes -O2 -fomit-frame-pointer
 
 AS         = $(CROSS_COMPILE)as
 LD         = $(CROSS_COMPILE)ld
diff -r c4512592a1dc -r 41ceeb6828b5 xen/tools/Makefile
--- a/xen/tools/Makefile        Mon Aug  8 08:18:38 2005
+++ b/xen/tools/Makefile        Mon Aug  8 08:32:52 2005
@@ -10,4 +10,4 @@
        rm -f *.o symbols
 
 symbols: symbols.c
-       $(HOSTCC) -o $@ $<
+       $(HOSTCC) $(HOSTCFLAGS) -o $@ $<
diff -r c4512592a1dc -r 41ceeb6828b5 xen/tools/symbols.c
--- a/xen/tools/symbols.c       Mon Aug  8 08:18:38 2005
+++ b/xen/tools/symbols.c       Mon Aug  8 08:32:52 2005
@@ -152,8 +152,8 @@
        /* include the type field in the symbol name, so that it gets
         * compressed together */
        s->len = strlen(str) + 1;
-       s->sym = (char *) malloc(s->len + 1);
-       strcpy(s->sym + 1, str);
+       s->sym = (unsigned char *) malloc(s->len + 1);
+       strcpy((char *)s->sym + 1, str);
        s->sym[0] = s->type;
 
        return 0;
@@ -197,16 +197,16 @@
                 * move then they may get dropped in pass 2, which breaks the
                 * symbols rules.
                 */
-               if (s->addr == _etext && strcmp(s->sym + offset, "_etext"))
+               if (s->addr == _etext && strcmp((char *)s->sym + offset, 
"_etext"))
                        return 0;
        }
 
        /* Exclude symbols which vary between passes. */
-       if (strstr(s->sym + offset, "_compiled."))
+       if (strstr((char *)s->sym + offset, "_compiled."))
                return 0;
 
        for (i = 0; special_symbols[i]; i++)
-               if( strcmp(s->sym + offset, special_symbols[i]) == 0 )
+               if( strcmp((char *)s->sym + offset, special_symbols[i]) == 0 )
                        return 0;
 
        return 1;

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog