afs: Remove dead assignments 22/15222/5
authorKailas Zadbuke <kailashsz@in.ibm.com>
Wed, 14 Dec 2022 14:00:24 +0000 (19:00 +0530)
committerBenjamin Kaduk <kaduk@mit.edu>
Thu, 5 Jan 2023 16:00:41 +0000 (11:00 -0500)
commit39beef42ab3020bbdc74b9594e6bb5c07f4d8522
tree75dc8d9a333136c9ae948c4b8d4c7372edc61343
parentcfac0df9cd7152be2672c665442aac84215494d6
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
src/afs/afs_dcache.c