clang-13: remove unused variables flagged by clang 75/14775/4
authorCheyenne Wills <cwills@sinenomine.net>
Fri, 27 Aug 2021 14:20:42 +0000 (08:20 -0600)
committerBenjamin Kaduk <kaduk@mit.edu>
Wed, 1 Sep 2021 23:58:13 +0000 (19:58 -0400)
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
src/afs/VNOPS/afs_vnop_write.c
src/libadmin/vos/vsprocs.c
src/ubik/disk.c
src/ubik/remote.c

index e098278..27065e6 100644 (file)
@@ -46,7 +46,6 @@ afs_read(struct vcache *avc, struct uio *auio, afs_ucred_t *acred,
         int noLock)
 {
     afs_size_t totalLength;
-    afs_size_t transferLength;
     afs_size_t filePos;
     afs_size_t offset, tlen;
     afs_size_t len = 0;
@@ -96,7 +95,6 @@ afs_read(struct vcache *avc, struct uio *auio, afs_ucred_t *acred,
               totalLength, ICL_TYPE_OFFSET,
               ICL_HANDLE_OFFSET(avc->f.m.Length));
     error = 0;
-    transferLength = 0;
     if (!noLock)
        ObtainReadLock(&avc->lock);
 #if    defined(AFS_TEXT_ENV) && !defined(AFS_VM_RDWR_ENV)
@@ -352,7 +350,6 @@ afs_read(struct vcache *avc, struct uio *auio, afs_ucred_t *acred,
        trimlen = len;
        afsio_skip(auio, trimlen);      /* update input uio structure */
        totalLength -= len;
-       transferLength += len;
        filePos += len;
 
        if (len <= 0)
index 90ad86c..cf8e334 100644 (file)
@@ -198,7 +198,6 @@ afs_write(struct vcache *avc, struct uio *auio, int aio,
             afs_ucred_t *acred, int noLock)
 {
     afs_size_t totalLength;
-    afs_size_t transferLength;
     afs_size_t filePos;
     afs_size_t offset, len;
     afs_int32 tlen;
@@ -232,7 +231,6 @@ afs_write(struct vcache *avc, struct uio *auio, int aio,
     totalLength = AFS_UIO_RESID(auio);
     filePos = AFS_UIO_OFFSET(auio);
     error = 0;
-    transferLength = 0;
     afs_Trace4(afs_iclSetp, CM_TRACE_WRITE, ICL_TYPE_POINTER, avc,
               ICL_TYPE_OFFSET, ICL_HANDLE_OFFSET(filePos), ICL_TYPE_OFFSET,
               ICL_HANDLE_OFFSET(totalLength), ICL_TYPE_OFFSET,
@@ -358,7 +356,6 @@ afs_write(struct vcache *avc, struct uio *auio, int aio,
                tdc->validPos = filePos + len;
        }
        totalLength -= len;
-       transferLength += len;
        filePos += len;
 #if defined(AFS_SGI_ENV)
        /* afs_xwrite handles setting m.Length */
index f22f42f..facd3d5 100644 (file)
@@ -3490,7 +3490,7 @@ UV_SyncServer(afs_cell_handle_p cellHandle, struct rx_connection *server,
     int rc = 0;
     afs_status_t tst = 0;
     int noError;
-    afs_int32 nentries, tentries = 0;
+    afs_int32 nentries;
     struct VldbListByAttributes attributes;
     nbulkentries arrayEntries;
     int totalF;
@@ -3519,7 +3519,6 @@ UV_SyncServer(afs_cell_handle_p cellHandle, struct rx_connection *server,
            goto fail_UV_SyncServer;
        }
        nsi = -1;
-       tentries += nentries;
        totalF = 0;
        for (j = 0; j < nentries; j++) {        /* process each entry */
            vllist = &arrayEntries.nbulkentries_val[j];
index 5b74369..0ef106f 100644 (file)
@@ -680,11 +680,10 @@ udisk_read(struct ubik_trans *atrans, afs_int32 afile, void *abuffer,
           afs_int32 apos, afs_int32 alen)
 {
     char *bp;
-    afs_int32 offset, len, totalLen;
+    afs_int32 offset, len;
 
     if (atrans->flags & TRDONE)
        return UDONE;
-    totalLen = 0;
     while (alen > 0) {
        bp = DRead(atrans, afile, apos >> UBIK_LOGPAGESIZE);
        if (!bp)
@@ -698,7 +697,6 @@ udisk_read(struct ubik_trans *atrans, afs_int32 afile, void *abuffer,
        abuffer = (char *)abuffer + len;
        apos += len;
        alen -= len;
-       totalLen += len;
        DRelease(bp, 0);
     }
     return 0;
@@ -746,7 +744,7 @@ udisk_write(struct ubik_trans *atrans, afs_int32 afile, void *abuffer,
            afs_int32 apos, afs_int32 alen)
 {
     char *bp;
-    afs_int32 offset, len, totalLen;
+    afs_int32 offset, len;
     struct ubik_trunc *tt;
     afs_int32 code;
 
@@ -769,7 +767,6 @@ udisk_write(struct ubik_trans *atrans, afs_int32 afile, void *abuffer,
     }
 
     /* now update vm */
-    totalLen = 0;
     while (alen > 0) {
        bp = DRead(atrans, afile, apos >> UBIK_LOGPAGESIZE);
        if (!bp) {
@@ -786,7 +783,6 @@ udisk_write(struct ubik_trans *atrans, afs_int32 afile, void *abuffer,
        abuffer = (char *)abuffer + len;
        apos += len;
        alen -= len;
-       totalLen += len;
        DRelease(bp, 1);        /* buffer modified */
     }
     return 0;
index 6874c6a..66d2318 100644 (file)
@@ -472,7 +472,6 @@ SDISK_SendFile(struct rx_call *rxcall, afs_int32 file,
     afs_int32 code;
     struct ubik_dbase *dbase = NULL;
     char tbuffer[1024];
-    afs_int32 offset;
     struct ubik_version tversion;
     int tlen;
     struct rx_peer *tpeer;
@@ -526,7 +525,6 @@ SDISK_SendFile(struct rx_call *rxcall, afs_int32 file,
     ViceLog(0, ("Ubik: Synchronize database: receive (via SendFile) from server %s begin\n",
               afs_inet_ntoa_r(otherHost, hoststr)));
 
-    offset = 0;
     UBIK_VERSION_LOCK;
     epoch = tversion.epoch = 0;                /* start off by labelling in-transit db as invalid */
     (*dbase->setlabel) (dbase, file, &tversion);       /* setlabel does sync */
@@ -569,7 +567,6 @@ SDISK_SendFile(struct rx_call *rxcall, afs_int32 file,
            close(fd);
            goto failed;
        }
-       offset += tlen;
        length -= tlen;
     }
     code = close(fd);