sim: igen: fix format-zero-length warnings
authorMike Frysinger <vapier@gentoo.org>
Tue, 9 Jan 2024 02:15:51 +0000 (21:15 -0500)
committerMike Frysinger <vapier@gentoo.org>
Tue, 9 Jan 2024 02:25:55 +0000 (21:25 -0500)
Fix warnings from calling printf functions with "" which normally
is useless.

sim/igen/gen.c

index 2176b60..30cf107 100644 (file)
@@ -67,7 +67,11 @@ print_gen_entry_path (const line_ref *line,
       if (table->top->model != NULL)
        print (line, "%s", table->top->model->name);
       else
-       print (line, "");
+       {
+         /* We don't want to output things, but we want the side-effects they
+            might have (e.g. checking line != NULL).  */
+         print (line, "%s", "");
+       }
     }
   else
     {
@@ -1242,7 +1246,7 @@ gen_entry_expand_insns (gen_entry *table)
          print_gen_entry_insns (table, warning,
                                 "was not uniquely decoded",
                                 "decodes to the same entry");
-         error (NULL, "");
+         error (NULL, "unrecoverable\n");
        }
       return;
     }
@@ -1385,7 +1389,7 @@ gen_entry_expand_insns (gen_entry *table)
                warning (NULL,
                         ": Applying rule just copied all instructions\n");
                print_gen_entry_insns (entry, warning, "Copied", NULL);
-               error (NULL, "");
+               error (NULL, "unrecoverable\n");
              }
          }
       }