sim: Remove self-assignments
authorTsukasa OI <research_trasio@irq.a4lg.com>
Sun, 25 Sep 2022 08:42:02 +0000 (08:42 +0000)
committerAndrew Burgess <aburgess@redhat.com>
Tue, 11 Oct 2022 14:18:14 +0000 (15:18 +0100)
Clang generates a warning if there is a redundant self-assignment
("-Wself-assign").  On the default configuration, it causes a build failure
(unless "--disable-werror" is specified).

This commit removes redundant self-assignments from two files.

sim/common/hw-tree.c
sim/ppc/tree.c

index 56319333d76f9975d888b490826a395e23959885..8bb5ac7754500a430eff51036dd9e6b32db0debe 100644 (file)
@@ -335,7 +335,6 @@ split_find_device (struct hw *current,
       else if (strncmp (spec->path, "./", strlen ("./")) == 0)
        {
          /* cd ./... */
-         current = current;
          spec->path += strlen ("./");
        }
       else if (strncmp (spec->path, "../", strlen ("../")) == 0)
@@ -348,7 +347,6 @@ split_find_device (struct hw *current,
       else if (strcmp (spec->path, ".") == 0)
        {
          /* cd . */
-         current = current;
          spec->path += strlen (".");
        }
       else if (strcmp (spec->path, "..") == 0)
index 6d20665505ea4b9b8909b3a310771f18c880ef59..05532bb47ee85cf00692debe836c143c44809b68 100644 (file)
@@ -306,7 +306,6 @@ split_find_device(device *current,
     }
     else if (strncmp(spec->path, "./", strlen("./")) == 0) {
       /* cd ./... */
-      current = current;
       spec->path += strlen("./");
     }
     else if (strncmp(spec->path, "../", strlen("../")) == 0) {
@@ -317,7 +316,6 @@ split_find_device(device *current,
     }
     else if (strcmp(spec->path, ".") == 0) {
       /* cd . */
-      current = current;
       spec->path += strlen(".");
     }
     else if (strcmp(spec->path, "..") == 0) {