[binutils-gdb] sim: options: fix --help output

Michael Frysinger vapier@sourceware.org
Sun May 2 03:09:44 GMT 2021


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=e4821e2f5fd281eb6b88b642fa9beaf9c3a35070

commit e4821e2f5fd281eb6b88b642fa9beaf9c3a35070
Author: Mike Frysinger <vapier@gentoo.org>
Date:   Sat May 1 23:04:01 2021 -0400

    sim: options: fix --help output
    
    The hash table rewrite broke --help output due to subtle behavior:
    calling dup_arg_p(NULL) will create & clear the table, not just
    create it.  The --help output relies on this to clear the table
    before it shows things.

Diff:
---
 sim/common/ChangeLog     | 4 ++++
 sim/common/sim-options.c | 1 +
 2 files changed, 5 insertions(+)

diff --git a/sim/common/ChangeLog b/sim/common/ChangeLog
index 55d684bfeab..7abaeb3a816 100644
--- a/sim/common/ChangeLog
+++ b/sim/common/ChangeLog
@@ -1,3 +1,7 @@
+2021-05-01  Mike Frysinger  <vapier@gentoo.org>
+
+	* sim-options.c (dup_arg_p): Call htab_empty.
+
 2021-05-01  Mike Frysinger  <vapier@gentoo.org>
 
 	* dv-sockser.c (dv_sockser_install): Rename to ...
diff --git a/sim/common/sim-options.c b/sim/common/sim-options.c
index a6da8eccdef..1522cac9379 100644
--- a/sim/common/sim-options.c
+++ b/sim/common/sim-options.c
@@ -437,6 +437,7 @@ dup_arg_p (const char *arg)
 	arg_table = htab_create_alloc (10, htab_hash_string,
 				       compare_strings, NULL,
 				       xcalloc, free);
+      htab_empty (arg_table);
       return 0;
     }


More information about the Gdb-cvs mailing list