The following commit has been merged in the master branch:
commit 39beef42ab3020bbdc74b9594e6bb5c07f4d8522
Author: Kailas Zadbuke <[email protected]>
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([email protected]) for working with me on this
patch.
Change-Id: Ic6f72e3f8b0f0a69c9bf3dfa064db76423620796
Signed-off-by: Kailas Zadbuke <[email protected]>
Reviewed-on: https://gerrit.openafs.org/15222
Reviewed-by: Andrew Deason <[email protected]>
Tested-by: Andrew Deason <[email protected]>
Reviewed-by: Cheyenne Wills <[email protected]>
Reviewed-by: Benjamin Kaduk <[email protected]>
src/afs/afs_bypasscache.c | 5 +----
src/afs/afs_dcache.c | 4 ----
2 files changed, 1 insertion(+), 8 deletions(-)
--
OpenAFS Master Repository
_______________________________________________
OpenAFS-cvs mailing list
[email protected]
https://lists.openafs.org/mailman/listinfo/openafs-cvs
