OpenAFS Master Repository branch, openafs-stable-1_8_x, updated. openafs-stable-1_8_9-43-g973c5c5
Gerrit Code Review
gerrit@openafs.org
Thu, 26 Jan 2023 07:19:01 -0500
The following commit has been merged in the openafs-stable-1_8_x branch:
commit 973c5c585f59c0e88a2852d290abcb5e8fdf2b86
Author: Cheyenne Wills <cwills@sinenomine.net>
Date: Thu Dec 1 12:09:09 2022 -0700
clang-16: Fix simple unused-but-set variables
Clang-16 is flagging variables as unused-but-set-variables resulting in
build errors when --enable-warning is used.
db_verify.c:996:9: error: variable 'builtinUsers' set but not used
[-Werror,-Wunused-but-set-variable]
int builtinUsers = 0;
^
Remove the variables that are set but never actually used.
This commit fixes the simpler cases where variables are only set but
not used elsewhere.
Reviewed-on: https://gerrit.openafs.org/15178
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
(cherry picked from commit afafde158ff875d2459bb81ac373cde3e45dac03)
Change-Id: I65cd27325b768ba192831401f4ac71e94071ecc7
Reviewed-on: https://gerrit.openafs.org/15273
Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
Reviewed-by: Harish Sharma <Harish.Sharma1@ibm.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>
src/afs/afs_user.c | 3 ---
src/afs/afs_vcache.c | 2 --
src/bu_utils/fms.c | 5 -----
src/bucoord/main.c | 3 ---
src/libadmin/vos/vsprocs.c | 20 ++------------------
src/ptserver/db_verify.c | 5 +----
src/vlserver/vlclient.c | 4 +---
src/volser/dumpstuff.c | 3 ---
8 files changed, 4 insertions(+), 41 deletions(-)
--
OpenAFS Master Repository