Update warning management
authorSimon Wilkinson <sxw@inf.ed.ac.uk>
Fri, 16 Oct 2009 21:45:50 +0000 (22:45 +0100)
committerDerrick Brashear <shadow|account-1000005@unknown>
Sun, 18 Oct 2009 13:13:58 +0000 (06:13 -0700)
This change tidies up after the recent slew of warning reduction. It
updates README.WARNINGS to reflect the current state of the tree,
disables warnings for a couple of files that are now warning clean,
and ensures that the libuafs and libafsrpc Makefiles match the state
of the rest of the tree in terms of which warnings are enabled.

Reviewed-on: http://gerrit.openafs.org/672
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>

README.WARNINGS
src/fsprobe/Makefile.in
src/kauth/Makefile.in
src/libafsrpc/Makefile.in
src/libuafs/Makefile.common.in
src/tviced/Makefile.in
src/volser/vsprocs.c

index 79a9c43..c954710 100644 (file)
@@ -72,7 +72,6 @@ libadmin/bos/afs_bosAdmin.c: all     : DES keys, time types, and const
 libadmin/kas/afs_kasAdmin.c: all     : Ubik_Call nonsense, DES keys
 libadmin/samples/rxstat_query_peer.c : all : util_RPCStatsStateGet types
 libadmin/samples/rxstat_query_process.c : all : util_RPCStatsStateGet types
-libadmin/samples/rxdebug_peers.c : format : struct clock vs unsigned int
 libadmin/test/client.c : all         : util_RPCStatsStateGet types
 ptserver/ptserver.c  : all          : ExtendedCellInfo
 rxkad/rxkad_server.c : all           : Des key nonsense
@@ -88,10 +87,10 @@ ubik/ubikclient.c    : strict-protos : ubik_Call
 uss/uss_vol.c       : all             Format issues
 uss/uss_kauth.c      : all          : Des keys, ubik_Call
 uss/lex.yy.c         : all           : Unused symbols
+viced/afsfileprocs.c : all (tviced)  : implicit definiton of hpr_*
 viced/fsprobe.c      : all          : ubik_Call
 vlserver/vlserver.c  : all          : GetExtendedCellInfo
 vol/salvager.c       : all           : consts & undefined variable from header
 volser/vos.c         : all          : ubik_Call_New
 volser/vol-dump.c    : format        : afs_sfsize_t
-volser/vsprocs.c     : format-extra  : The ONERROR macro needs to be rethought
 
index 0550f5c..c9e6bd4 100644 (file)
@@ -46,7 +46,6 @@ libfsprobe.a: fsprobe.o fsprobe_callback.o AFS_component_version_number.o
        $(RANLIB) $@
 
 fsprobe.o: fsprobe.c ${INCLS} AFS_component_version_number.c
-       $(CC) $(CFLAGS) @CFLAGS_NOERROR@ -c $<
 
 fsprobe_callback.o: fsprobe_callback.c ${INCLS}
 
index 7b81fba..c408f47 100644 (file)
@@ -101,7 +101,6 @@ kaprocs.o: kaprocs.c ${INCLS} kaport.h
        ${CC} ${CFLAGS} @CFLAGS_NOERROR@ -c $<
 
 authclient.o: authclient.c ${INCLS}
-       $(CC) $(CFLAGS) @CFLAGS_NOERROR@ -c $<
 
 kaauxdb.o: kaauxdb.c
        $(CC) $(CFLAGS) @CFLAGS_NOERROR@ -c $<
index 50c8aa1..6796a66 100644 (file)
@@ -199,7 +199,7 @@ rxkad_common.o: ${RXKAD}/rxkad_common.c
        ${CCRULE} ${RXKAD}/rxkad_common.c
 
 ticket.o: ${RXKAD}/ticket.c
-       ${CCRULE} @CFLAGS_NOERROR@ ${RXKAD}/ticket.c
+       ${CCRULE} ${RXKAD}/ticket.c
 
 ticket5.o: ${RXKAD}/ticket5.c
        ${CCRULE} @CFLAGS_NOERROR@ ${RXKAD}/ticket5.c
index d6f777d..30d52d4 100644 (file)
@@ -788,7 +788,7 @@ $(UOBJ)/osi_vfsops.o: $(TOP_SRC_AFS)/UKERNEL/osi_vfsops.c
 $(UOBJ)/ktc.o:         $(TOP_SRCDIR)/auth/ktc.c
        $(CRULE1) @CFLAGS_NOERROR@
 $(UOBJ)/ticket.o: $(TOP_SRCDIR)/rxkad/ticket.c
-       $(CRULE1) @CFLAGS_NOERROR@
+       $(CRULE1)
 $(UOBJ)/rxkad_server.o: $(TOP_SRCDIR)/rxkad/rxkad_server.c
        $(CRULE1)
 $(UOBJ)/ptint.cs.o: $(TOP_OBJDIR)/src/ptserver/ptint.cs.c
@@ -1056,7 +1056,7 @@ $(WEBOBJ)/securehash.o: $(TOP_SRCDIR)/afsweb/securehash.c
 $(WEBOBJ)/nsafs.o: $(TOP_SRCDIR)/afsweb/nsafs.c
        $(CRULE2)
 $(WEBOBJ)/ticket.o: $(TOP_SRCDIR)/rxkad/ticket.c
-       $(CRULE2) @CFLAGS_NOERROR@
+       $(CRULE2)
 $(WEBOBJ)/rxkad_server.o: $(TOP_SRCDIR)/rxkad/rxkad_server.c
        $(CRULE2)
 $(WEBOBJ)/ptint.cs.o: $(TOP_OBJDIR)/src/ptserver/ptint.cs.c
@@ -1318,7 +1318,7 @@ $(JUAFS)/osi_vfsops.o: $(TOP_SRC_AFS)/UKERNEL/osi_vfsops.c
 $(JUAFS)/ktc.o:        $(TOP_SRCDIR)/auth/ktc.c
        $(CRULE1) @CFLAGS_NOERROR@
 $(JUAFS)/ticket.o: $(TOP_SRCDIR)/rxkad/ticket.c
-       $(CRULE1) @CFLAGS_NOERROR@
+       $(CRULE1)
 $(JUAFS)/rxkad_server.o: $(TOP_SRCDIR)/rxkad/rxkad_server.c
        $(CRULE1)
 $(JUAFS)/ptint.cs.o: $(TOP_OBJDIR)/src/ptserver/ptint.cs.c
index 4c01394..946415a 100644 (file)
@@ -68,7 +68,7 @@ afsfileprocs.o: ${VICED}/afsfileprocs.c
        ${CCRULE} @CFLAGS_NOERROR@
 
 host.o: ${VICED}/host.c
-       ${CCRULE} @CFLAGS_NOERROR@
+       ${CCRULE}
 
 physio.o: ${VICED}/physio.c
        ${CCRULE}
index 678effe..8adbceb 100644 (file)
 #include <afsconfig.h>
 #include <afs/param.h>
 
-#ifdef IGNORE_SOME_GCC_WARNINGS
-# pragma GCC diagnostic warning "-Wformat-extra-args"
-#endif
-
 #include <stdio.h>
 #include <sys/types.h>
 #include <sys/stat.h>