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