OpenAFS Master Repository branch, master, updated. openafs-devel-1_9_1-77-g8333e8e
Gerrit Code Review
gerrit@openafs.org
Wed, 1 Sep 2021 19:58:37 -0400
The following commit has been merged in the master branch:
commit 8333e8e6020331013af912acb92a308e4f5a1dd2
Author: Cheyenne Wills <cwills@sinenomine.net>
Date: Fri Aug 27 08:20:42 2021 -0600
clang-13: remove unused variables flagged by clang
Clang-13 changed the default for the unused-but-set-variable resulting
in build warnings/errors with the following type of messages
vsprocs.c:3493:25: error: variable 'tentries' set but not used
[-Werror,-Wunused-but-set-variable]
afs_int32 nentries, tentries = 0;
The locations where these local variables are being flagged show that
while the variables are being updated, they are actually never used for
anything (e.g. used as part of an assignment to another variable, passed
as a parameter, used for as returned value, etc.)
Remove the variables being flagged by the clang-13 compiler.
Removal of these variables will not alter the overall functionality of
the code.
Change-Id: I521ea1323837e0d293e6f45eb8c15d5e05765f19
Reviewed-on: https://gerrit.openafs.org/14775
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
src/afs/VNOPS/afs_vnop_read.c | 3 ---
src/afs/VNOPS/afs_vnop_write.c | 3 ---
src/libadmin/vos/vsprocs.c | 3 +--
src/ubik/disk.c | 8 ++------
src/ubik/remote.c | 3 ---
5 files changed, 3 insertions(+), 17 deletions(-)
--
OpenAFS Master Repository