OpenAFS Master Repository branch, master, updated. openafs-devel-1_9_1-228-g05eb420
Gerrit Code Review
gerrit@openafs.org
Thu, 1 Dec 2022 21:25:43 -0500
The following commit has been merged in the master branch:
commit 05eb420829f9bca155c1d5af9fe96844db974770
Author: Cheyenne Wills <cwills@sinenomine.net>
Date: Thu Dec 1 09:05:58 2022 -0700
clang-16: Fix unused-but-set 'for' control vars
clang-16 is flagging unused-but-set variables which result in build
errors when --enable-warning is turned on. Several of these variables
are used within 'for(..)' loops but are not actually referenced anywhere
else.
fcrypt.c:181:16: error: variable 'i' set but not used
[-Werror,-Wunused-but-set-variable]
afs_uint32 i, j;
^
Refactor the for statements to eliminate the unused variables.
Note, this commit is a little more complex than simply removing unused
variables.
Change-Id: I4d2de60dffc3e717d784e820f2d91470c5e7926b
Reviewed-on: https://gerrit.openafs.org/15176
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
src/rxdebug/rxdebug.c | 5 ++---
src/rxkad/fcrypt.c | 6 +++---
src/viced/callback.c | 4 ++--
3 files changed, 7 insertions(+), 8 deletions(-)
--
OpenAFS Master Repository