The following commit has been merged in the openafs-stable-1_8_x branch: commit 973c5c585f59c0e88a2852d290abcb5e8fdf2b86 Author: Cheyenne Wills <cwi...@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 <build...@rampaginggeek.com> Reviewed-by: Andrew Deason <adea...@sinenomine.net> Reviewed-by: Benjamin Kaduk <ka...@mit.edu> (cherry picked from commit afafde158ff875d2459bb81ac373cde3e45dac03) Change-Id: I65cd27325b768ba192831401f4ac71e94071ecc7 Reviewed-on: https://gerrit.openafs.org/15273 Reviewed-by: Michael Meffie <mmef...@sinenomine.net> Reviewed-by: Harish Sharma <harish.shar...@ibm.com> Tested-by: BuildBot <build...@rampaginggeek.com> Reviewed-by: Stephan Wiesand <stephan.wies...@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 _______________________________________________ OpenAFS-cvs mailing list OpenAFS-cvs@openafs.org https://lists.openafs.org/mailman/listinfo/openafs-cvs