OpenAFS Master Repository branch, master, updated. openafs-devel-1_9_1-229-g97018ac
Gerrit Code Review
gerrit@openafs.org
Thu, 1 Dec 2022 21:29:51 -0500
The following commit has been merged in the master branch:
commit 97018ace991d300a6ff75a1dcd1a69b4044f2b89
Author: Cheyenne Wills <cwills@sinenomine.net>
Date: Wed Nov 2 14:47:04 2022 -0600
clang-16: Fix conditionally unused-but-set variables
clang-16 is flagging unused-but-set variables which result in build
errors when --enable-warning is turned on.
remote.c:485:15: error: variable 'pass' set but not used
[-Werror,-Wunused-but-set-variable]
afs_int32 pass;
^
These variables are actually used in specific cases depending on
build configuration (e.g. AFS_PTHREAD_ENV, etc.).
Relocate these variables so they are fully defined or referenced within
preprocessor '#if' blocks.
Change-Id: I900a192ec781be4ab4453769e1d8f0cc48fe5757
Reviewed-on: https://gerrit.openafs.org/15177
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
src/afs/afs_daemons.c | 4 ++++
src/kauth/kadatabase.c | 6 ++++--
src/ubik/recovery.c | 6 +++++-
src/ubik/remote.c | 8 ++++----
src/vol/volume.c | 4 +++-
5 files changed, 20 insertions(+), 8 deletions(-)
--
OpenAFS Master Repository