gdb/options: fix copy&paste error in string_option_def
authorAndrew Burgess <aburgess@redhat.com>
Thu, 14 Dec 2023 11:54:13 +0000 (11:54 +0000)
committerAndrew Burgess <aburgess@redhat.com>
Thu, 14 Dec 2023 16:29:09 +0000 (16:29 +0000)
Spotted what appears to be a copy&paste error in string_option_def,
the code for string handling writes the address fetching callback
function into the option_def::var_address::enumeration location,
rather than option_def::var_address::string.

Of course, this works just fine as option_def::var_address is a union,
and all of its members are function pointers, so they're going to be
the same size on every target GDB cares about.

But it doesn't hurt to be correct, so fixed in this commit.

There should be no user visible changes after this commit.

gdb/cli/cli-option.h

index 4c62227..2645c58 100644 (file)
@@ -304,7 +304,7 @@ struct string_option_def : option_def
                  show_cmd_cb_,
                  set_doc_, show_doc_, help_doc_)
   {
-    var_address.enumeration = detail::get_var_address<const char *, Context>;
+    var_address.string = detail::get_var_address<std::string, Context>;
   }
 };