Retire the AFS_PTR_FMT macro 30/13830/6
authorCheyenne Wills <cwills@sinenomine.net>
Tue, 24 Sep 2019 21:59:47 +0000 (15:59 -0600)
committerBenjamin Kaduk <kaduk@mit.edu>
Fri, 27 Sep 2019 15:01:45 +0000 (11:01 -0400)
commita133f1b1e7eb605c36ac16a6ed115bef03e8a004
treef92c59e7561cb19513b6bd62ef77c972f9f3f597
parent0f1e54c47c179bdbd69799170d9740e3e58e86db
Retire the AFS_PTR_FMT macro

Originally '%x' was commonly used as the printf specifier for formatting
pointer values.

Commit 37fc3b01445cd6446f09c476ea2db47fea544b7d introduced the
AFS_PTR_FMT macro to support platform-dependent printf format specifiers
for pointer representation. This macro defined the format specifier as
'%p' for Windows, and '%x' for non-Windows platforms.

Commit 2cf12c43c6a5822212f1d4e42dca7c059a1a9000 changed the printf
pointer format specifier from '%x' to '%p' on non-Windows platforms as
well, so at this point '%p' is the printf pointer format specifier for
all supported platforms.

Since the AFS_PRT_FMT macro is no longer platform-dependent, and all C89
compilers support the '%p' specifier, retire the macro to simplify the
printf format strings.

Change-Id: I0cb13cccbe6a8d0000edd162b623ddcdb74c1cf7
Reviewed-on: https://gerrit.openafs.org/13830
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
16 files changed:
src/budb/database.c
src/budb/db_hash.c
src/budb/db_text.c
src/config/stds.h
src/dir/salvage.c
src/lwp/lwp.c
src/rx/rx.c
src/rx/rx_packet.c
src/viced/host.c
src/viced/state_analyzer.c
src/vol/clone.c
src/vol/vg_cache.c
src/vol/vnode.c
src/xstat/xstat_cm.c
src/xstat/xstat_fs.c
src/xstat/xstat_fs_test.c