OpenAFS Master Repository branch, master, updated. BP-openafs-stable-1_8_x-468-ga133f1b
Gerrit Code Review
gerrit@openafs.org
Fri, 27 Sep 2019 11:05:21 -0400
The following commit has been merged in the master branch:
commit a133f1b1e7eb605c36ac16a6ed115bef03e8a004
Author: Cheyenne Wills <cwills@sinenomine.net>
Date: Tue Sep 24 15:59:47 2019 -0600
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>
src/budb/database.c | 8 ++--
src/budb/db_hash.c | 2 +-
src/budb/db_text.c | 4 +-
src/config/stds.h | 2 -
src/dir/salvage.c | 20 +++++--------
src/lwp/lwp.c | 2 +-
src/rx/rx.c | 38 +++++++++++++-----------
src/rx/rx_packet.c | 14 ++++----
src/viced/host.c | 68 ++++++++++++++++++++++----------------------
src/viced/state_analyzer.c | 4 +-
src/vol/clone.c | 4 +-
src/vol/vg_cache.c | 6 ++--
src/vol/vnode.c | 7 ++--
src/xstat/xstat_cm.c | 6 ++--
src/xstat/xstat_fs.c | 6 ++--
src/xstat/xstat_fs_test.c | 2 +-
16 files changed, 94 insertions(+), 99 deletions(-)
--
OpenAFS Master Repository