diff options
| author | Tom Tromey <tromey@adacore.com> | 2019-03-06 07:55:42 -0700 |
|---|---|---|
| committer | Tom Tromey <tromey@adacore.com> | 2019-03-06 07:59:43 -0700 |
| commit | 67a3048c0fe804ab1e36d4074e7fd2dadeb8bc0b (patch) | |
| tree | 7e02a00c85d76f343e5a83797e3c1ad52cb1b05d /gdb/remote-sim.c | |
| parent | c3734e093aab1cea90a76b107cdda4a5870dd1b8 (diff) | |
| download | gdb-67a3048c0fe804ab1e36d4074e7fd2dadeb8bc0b.zip gdb-67a3048c0fe804ab1e36d4074e7fd2dadeb8bc0b.tar.gz gdb-67a3048c0fe804ab1e36d4074e7fd2dadeb8bc0b.tar.bz2 | |
Fix remote-sim.c build after warn-unused-result change
John Darrington pointed out that commit 18cb7c9f3 ("Introduce
ATTRIBUTE_UNUSED_RESULT and use it") broke the build:
/home/john/binutils-gdb/gdb/remote-sim.c: In function 'void gdbsim_target_open(const char*, int)':
/home/john/binutils-gdb/gdb/remote-sim.c:765:18: error: ignoring return value of 'char** gdb_argv::release()', declared with attribute warn_unused_result [-Werror=unused-result]
This patch fixes the problem by arranging to use the result of the
release method.
Tested by rebuilding with a simulator enabled.
gdb/ChangeLog
2019-03-06 Tom Tromey <tromey@adacore.com>
* remote-sim.c (gdbsim_target_open): Use result of
gdb_argv::release.
Diffstat (limited to 'gdb/remote-sim.c')
| -rw-r--r-- | gdb/remote-sim.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/gdb/remote-sim.c b/gdb/remote-sim.c index 2acfc71..bef04e6 100644 --- a/gdb/remote-sim.c +++ b/gdb/remote-sim.c @@ -751,19 +751,18 @@ gdbsim_target_open (const char *args, int from_tty) } gdb_argv argv (arg_buf); - sim_argv = argv.get (); + sim_argv = argv.release (); init_callbacks (); gdbsim_desc = sim_open (SIM_OPEN_DEBUG, &gdb_callback, exec_bfd, sim_argv); if (gdbsim_desc == 0) { + freeargv (sim_argv); sim_argv = NULL; error (_("unable to create simulator instance")); } - argv.release (); - /* Reset the pid numberings for this batch of sim instances. */ next_pid = INITIAL_PID; |