]> git.ipfire.org Git - thirdparty/man-pages.git/commitdiff
memusage.1: Use sizeof consistently
authorAlejandro Colomar <colomar.6.4.3@gmail.com>
Thu, 3 Sep 2020 10:23:52 +0000 (12:23 +0200)
committerMichael Kerrisk <mtk.manpages@gmail.com>
Fri, 4 Sep 2020 07:43:36 +0000 (09:43 +0200)
Hi Michael,

Continuing with the series, this is the first of the last set of
patches: (2).1 as numbered in previous emails.

Regards,
Alex.

------------------------------------------------------------------------
>From ad5f958ed68079791d6e35f9d70ca5ec2a72c43b Mon Sep 17 00:00:00 2001
From: Alejandro Colomar <colomar.6.4.3@gmail.com>
Date: Thu, 3 Sep 2020 12:11:18 +0200
Subject: [PATCH] memusage.1: Use sizeof consistently

Use ``sizeof`` consistently through all the examples in the following
way:

- Use the name of the variable instead of its type as argument for
  ``sizeof``.

Rationale:
https://www.kernel.org/doc/html/v5.8/process/coding-style.html#allocating-memory

Signed-off-by: Alejandro Colomar <colomar.6.4.3@gmail.com>
Signed-off-by: Michael Kerrisk <mtk.manpages@gmail.com>
man1/memusage.1

index fa1987c79d16a58a012be7813ffc5de2fe722b66..a034684426db2a80a2739fb86daf5e6f1bcff0d5 100644 (file)
@@ -247,8 +247,8 @@ main(int argc, char *argv[])
      int i, j;
      int *p;
 
-     printf("malloc: %zd\en", sizeof(int) * 100);
-     p = malloc(sizeof(int) * 100);
+     printf("malloc: %zd\en", sizeof(*p) * 100);
+     p = malloc(sizeof(*p) * 100);
 
      for (i = 0; i < CYCLES; i++) {
          if (i < CYCLES / 2)
@@ -256,11 +256,11 @@ main(int argc, char *argv[])
          else
              j--;
 
-         printf("realloc: %zd\en", sizeof(int) * (j * 50 + 110));
-         p = realloc(p, sizeof(int) * (j * 50 + 100));
+         printf("realloc: %zd\en", sizeof(*p) * (j * 50 + 110));
+         p = realloc(p, sizeof(*p) * (j * 50 + 100));
 
-         printf("realloc: %zd\en", sizeof(int) * ((j+1) * 150 + 110));
-         p = realloc(p, sizeof(int) * ((j + 1) * 150 + 110));
+         printf("realloc: %zd\en", sizeof(*p) * ((j+1) * 150 + 110));
+         p = realloc(p, sizeof(*p) * ((j + 1) * 150 + 110));
      }
 
      free(p);