ticket-2618-patches-20031207
authorJeffrey Altman <jaltman@grand.central.org>
Sun, 7 Dec 2003 22:49:00 +0000 (22:49 +0000)
committerJeffrey Altman <jaltman@secure-endpoints.com>
Sun, 7 Dec 2003 22:49:00 +0000 (22:49 +0000)
commit21caaae64f4565d1a7ed82e326ef852d4cd3642f
treec58aa4a99b41007dd5aad0ebdce946ba01d37b53
parent4ceaebdbda533374ab7c66a832096988379a3ec8
ticket-2618-patches-20031207

TICKET 2618

flexelint patches for prototype handling from Joe Beuhler

>>>>>>>>>>>>>>>>>>>>>>>>>>>>

This patch adds a bunch of missing prototypes, makes various
corrections to global prototypes, and removes global declarations that
are not needed.

One set of added prototypes is commented out because it applies to
some printf-like functions that are implemented the way it used to be
done before varargs existed, and they need to be unprototyped or the
code will fail to compile.  (There are a number of functions in the
OpenAFS code that need to be converted to use varargs...)

There is one place of note in src/WINNT/afsd/fs.c where typedefs for
afs_int32 and afs_uint32 conflict with afs/stds.h.  I just added a
note to that effect.

Prototypes in src/WINNT/afsd/fs.c are incorrect for Windows NT.  I
don't know whether the return type involved changes from platform to
platform -- they come from .xg files.

Some prototypes are now commented out or #ifdef'd to match the code
that uses them.

Some global variables conflict between modules and have been made static.
The bufferBlock variable was of two completely different types...

In src/config/stds.h, I changed the declaration of afs_intmax_t to
reflect the fact that the Windows platform supports 64 bit integers.
This may be incorrect semantically, but I believe it is needed for the
%llx format to match afs_intmax_t arguments (for printing of inodes
mainly, I believe).

There were two different declarations for the rxkad_stats structure --
I synced them up.

- Joe

Removed the conflicting typedefs from src/WINNT/afsd/fs.c

Removed the changes to src/config/stds.h.  Added declaration of
AFS_64BIT_CLIENT to src/config/NTMakefile.i386_nt40 instead.

Do not remove unused variables which are defintions of data type
values.  Instead comment them out to avoid space utilization and
warnings.
116 files changed:
src/WINNT/afsadmsvr/TaAfsAdmSvrMain.cpp
src/WINNT/afsapplib/fastlist.cpp
src/WINNT/afsclass/worker.cpp
src/WINNT/afsd/afsd_init.c
src/WINNT/afsd/afsd_service.c
src/WINNT/afsd/cm.h
src/WINNT/afsd/cm_buf.c
src/WINNT/afsd/cm_callback.c
src/WINNT/afsd/cm_dcache.c
src/WINNT/afsd/cm_dns.c
src/WINNT/afsd/cm_freelance.c
src/WINNT/afsd/cm_ioctl.c
src/WINNT/afsd/cm_rpc.c
src/WINNT/afsd/cm_scache.c
src/WINNT/afsd/cm_vnodeops.c
src/WINNT/afsd/fs.c
src/WINNT/afsd/smb.c
src/WINNT/afsd/smb.h
src/WINNT/afsd/smb3.c
src/WINNT/afsd/smb_ioctl.c
src/WINNT/afssvrcfg/config_server_page.cpp
src/WINNT/afssvrcfg/get_cur_config.cpp
src/WINNT/afssvrcfg/partitions_page.cpp
src/WINNT/afssvrcfg/salvage_dlg.cpp
src/WINNT/afssvrcfg/services_page.cpp
src/WINNT/afssvrcfg/validation.cpp
src/WINNT/afssvrcfg/volume_utils.cpp
src/WINNT/afssvrmgr/dispguts.cpp
src/WINNT/afssvrmgr/task.cpp
src/WINNT/afssvrmgr/window.cpp
src/WINNT/afsusrmgr/mch_delete.cpp
src/WINNT/afsusrmgr/task.cpp
src/WINNT/client_creds/main.cpp
src/WINNT/install/NSIS/MakeCommon.bat
src/auth/userok.c
src/auth/writeconfig.c
src/bozo/bnode.p.h
src/bozo/bos.c
src/bozo/bosoprocs.c
src/bozo/bosserver.c
src/bozo/cronbnodeops.c
src/bozo/fsbnodeops.c
src/bucoord/bc_status.c
src/bucoord/commands.c
src/bucoord/dlq.c
src/bucoord/dsvs.c
src/bucoord/dump.c
src/bucoord/main.c
src/bucoord/restore.c
src/bucoord/vol_sets.c
src/bucoord/volstub.c
src/budb/database.h
src/budb/procs.c
src/budb/server.c
src/butc/butc_xbsa.c
src/butc/dbentries.c
src/butc/dump.c
src/butc/error_macros.h
src/butc/list.c
src/butc/lwps.c
src/butc/recoverDb.c
src/butc/tcprocs.c
src/config/NTMakefile.i386_nt40
src/config/mkvers.c
src/des/new_rnd_key.c
src/des/read_pssword.c
src/des/stats.h
src/des/test/testit.c
src/des/test/verify.c
src/dir/buffer.c
src/dir/dir.h
src/dir/salvage.c
src/kauth/admin_tools.c
src/kauth/kadatabase.c
src/kauth/kaserver.c
src/kauth/kautils.p.h
src/kauth/kpasswd.c
src/kauth/krb_udp.c
src/libadmin/bos/afs_bosAdmin.c
src/libadmin/kas/afs_kasAdmin.c
src/libadmin/pts/afs_ptsAdmin.c
src/libadmin/test/client.c
src/libadmin/vos/afs_vosAdmin.c
src/libadmin/vos/vosutils.c
src/lwp/fasttime.c
src/lwp/lwp_nt.c
src/lwp/timer.c
src/ptserver/ptclient.c
src/ptserver/testpt.c
src/ptserver/ubik.c
src/rx/rx_clock_nt.c
src/rx/rxdebug.c
src/rxkad/rxkad_client.c
src/rxkad/rxkad_common.c
src/ubik/disk.c
src/ubik/lock.c
src/ubik/ubik.p.h
src/ubik/udebug.c
src/update/server.c
src/util/regex.c
src/util/serverLog.c
src/viced/physio.c
src/vlserver/vlclient.c
src/vlserver/vlserver.c
src/vlserver/vlserver.p.h
src/vlserver/vlutils.c
src/vol/fssync.c
src/vol/ihandle.c
src/vol/ntops.c
src/vol/vol-salvage.c
src/volser/volmain.c
src/volser/volprocs.c
src/volser/volser.p.h
src/volser/vos.c
src/volser/vsprocs.c
src/volser/vsutils.c