OpenAFS Master Repository branch, master, updated. openafs-devel-1_9_1-241-g39beef4

Gerrit Code Review gerrit@openafs.org
Thu, 5 Jan 2023 11:01:11 -0500


The following commit has been merged in the master branch:
commit 39beef42ab3020bbdc74b9594e6bb5c07f4d8522
Author: Kailas Zadbuke <kailashsz@in.ibm.com>
Date:   Wed Dec 14 19:30:24 2022 +0530

    afs: Remove dead assignments
    
    Clang scan-build reported variables being assigned value but not used.
    
    In afs_NoCacheFetchProc() in file afs_bypasscache.c, delete the early
    unused assignment of 'pp' and move the declaration of 'pp' closer to
    where it's used. Also, 'ciov' is now unused and should be deleted.
    
    In afs_GetDCache() in file src/afs/afs_dcache.c, value stored at line
    2562 in 'code' is never read. Hence we can remove it. Also
    value stored in variable 'i' is not used, so we can delete the
    following line.
            i = osi_Time();
    
    Also in afs_dcacheInit(), in file src/afs/afs_dcache.c, value stored in
    variable 'ablocks' is not used, hence we can delete the following line.
            ablocks = afiles * (AFS_FIRSTCSIZE / 1024);
    
    Thanks Indira Sawant(indira.sawant@ibm.com) for working with me on this
    patch.
    
    Change-Id: Ic6f72e3f8b0f0a69c9bf3dfa064db76423620796
    Signed-off-by: Kailas Zadbuke <kailashsz@in.ibm.com>
    Reviewed-on: https://gerrit.openafs.org/15222
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Tested-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Cheyenne Wills <cwills@sinenomine.net>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

 src/afs/afs_bypasscache.c | 5 +----
 src/afs/afs_dcache.c      | 4 ----
 2 files changed, 1 insertion(+), 8 deletions(-)

-- 
OpenAFS Master Repository