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