The following commit has been merged in the openafs-stable-1_8_x branch:
commit 973c5c585f59c0e88a2852d290abcb5e8fdf2b86
Author: Cheyenne Wills <[email protected]>
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 <[email protected]>
    Reviewed-by: Andrew Deason <[email protected]>
    Reviewed-by: Benjamin Kaduk <[email protected]>
    (cherry picked from commit afafde158ff875d2459bb81ac373cde3e45dac03)
    
    Change-Id: I65cd27325b768ba192831401f4ac71e94071ecc7
    Reviewed-on: https://gerrit.openafs.org/15273
    Reviewed-by: Michael Meffie <[email protected]>
    Reviewed-by: Harish Sharma <[email protected]>
    Tested-by: BuildBot <[email protected]>
    Reviewed-by: Stephan Wiesand <[email protected]>

 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
[email protected]
https://lists.openafs.org/mailman/listinfo/openafs-cvs

Reply via email to