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