]> granicus.if.org Git - graphviz/commitdiff
remove unnecessary casts from parameters to memcpy
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Thu, 22 Oct 2020 00:05:05 +0000 (17:05 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Thu, 29 Oct 2020 00:13:12 +0000 (17:13 -0700)
These implicitly coerce to void* so a cast is not required.

lib/sfio/sfflsbuf.c
lib/sfio/sfmode.c
lib/sfio/sfungetc.c
lib/sparse/SparseMatrix.c

index e7e81456e73c31da89340111cc7a4fe74801fa0c..7c72a1e8d65d33b3bb935c6fb2b7d7acfb343ad9 100644 (file)
@@ -80,7 +80,7 @@ int _sfflsbuf(Sfio_t * f, int c)
        isall = SFISALL(f, isall);
        if ((w = SFWR(f, data, n, f->disc)) > 0) {
            if ((n -= w) > 0)   /* save unwritten data, then resume */
-               memcpy((char *) f->data, (char *) data + w, n);
+               memcpy(f->data, (char *) data + w, n);
            f->next = f->data + n;
            if (c < 0 && (!isall || n == 0))
                break;
index 0cc883ee14365f011a361a2a366c1edcaab5963c..85fc83b1ba0b895e75562a0b3692b9d956aed0fa 100644 (file)
@@ -109,8 +109,7 @@ int _sfsetpool(Sfio_t * f)
                goto done;
 
            /* move old array to new one */
-           memcpy((void *) array, (void *) p->sf,
-                  p->n_sf * sizeof(Sfio_t *));
+           memcpy(array, p->sf, p->n_sf * sizeof(Sfio_t *));
            if (p->sf != p->array)
                free(p->sf);
 
@@ -262,13 +261,13 @@ static int _sfpmode(Sfio_t * f, int type)
            }
        }
        if (p->ndata > 0)
-           memcpy((void *) p->rdata, (void *) f->next, p->ndata);
+           memcpy(p->rdata, f->next, p->ndata);
        f->endb = f->data;
     } else {                   /* restore read data */
        if (p->ndata > f->size) /* may lose data!!! */
            p->ndata = f->size;
        if (p->ndata > 0) {
-           memcpy((void *) f->data, (void *) p->rdata, p->ndata);
+           memcpy(f->data, p->rdata, p->ndata);
            f->endb = f->data + p->ndata;
            p->ndata = 0;
        }
index add8a1962767e56daf9769a50a4793c37a841177..31f79d10567f9716f2181bded11898c1663f65bc 100644 (file)
@@ -78,7 +78,7 @@ int sfungetc(Sfio_t * f, int c)
        }
        f->flags |= SF_MALLOC;
        if (f->data)
-           memcpy((char *) (data + 16), (char *) f->data, f->size);
+           memcpy(data + 16, f->data, f->size);
        f->size += 16;
        f->data = data;
        f->next = data + 16;
index ac68b36a4caafeb3792a350176ae76a27a07b878..19fe5f7707b411240a32bf43e7f1fd5c388ce4cd 100644 (file)
@@ -2988,7 +2988,7 @@ SparseMatrix SparseMatrix_get_augmented(SparseMatrix A){
     assert(A->size != 0 && nz > 0);
     val = MALLOC(A->size*2*((size_t)nz));
     memcpy(val, A->a, A->size*((size_t)nz));
-    memcpy((void*)(((char*) val) + ((size_t)nz)*A->size), A->a, A->size*((size_t)nz));
+    memcpy(((char*) val) + ((size_t)nz)*A->size, A->a, A->size*((size_t)nz));
   }
 
   nz = 0;