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

Reply via email to