diff options
author | Tom Tromey <tromey@adacore.com> | 2019-10-31 16:46:16 -0600 |
---|---|---|
committer | Tom Tromey <tromey@adacore.com> | 2019-11-01 10:34:21 -0600 |
commit | 5df96a4e6e8f1521274acb5beb54063c35aeec6e (patch) | |
tree | 6bfc777b7df9c85cce9cdb671acb9e54703b9f02 /gdb | |
parent | b74816496d5113596dd80bd071ed9bcf152c96cd (diff) | |
download | gdb-5df96a4e6e8f1521274acb5beb54063c35aeec6e.zip gdb-5df96a4e6e8f1521274acb5beb54063c35aeec6e.tar.gz gdb-5df96a4e6e8f1521274acb5beb54063c35aeec6e.tar.bz2 |
Simplify print_sys_errmsg
On irc, Christian pointed out that print_sys_errmsg could be
simplified by avoiding alloca entirely. This patch implements this.
gdb/ChangeLog
2019-11-01 Tom Tromey <tromey@adacore.com>
* utils.c (print_sys_errmsg): Simplify.
Change-Id: Ic399dade274ea61b63ef0540b3a3be2f0f80160a
Diffstat (limited to 'gdb')
-rw-r--r-- | gdb/ChangeLog | 4 | ||||
-rw-r--r-- | gdb/utils.c | 7 |
2 files changed, 5 insertions, 6 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 500c486..34e332b 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,9 @@ 2019-11-01 Tom Tromey <tromey@adacore.com> + * utils.c (print_sys_errmsg): Simplify. + +2019-11-01 Tom Tromey <tromey@adacore.com> + * gdbsupport/mingw-strerror.c (safe_strerror): Constify result. 2019-11-01 Christian Biesinger <cbiesinger@google.com> diff --git a/gdb/utils.c b/gdb/utils.c index e06eedd..f7fae35 100644 --- a/gdb/utils.c +++ b/gdb/utils.c @@ -629,15 +629,10 @@ void print_sys_errmsg (const char *string, int errcode) { const char *err = safe_strerror (errcode); - char *combined = (char *) alloca (strlen (err) + strlen (string) + 3); - strcpy (combined, string); - strcat (combined, ": "); - strcat (combined, err); - /* We want anything which was printed on stdout to come out first, before this message. */ gdb_flush (gdb_stdout); - fprintf_unfiltered (gdb_stderr, "%s.\n", combined); + fprintf_unfiltered (gdb_stderr, "%s: %s.\n", string, err); } /* Control C eventually causes this to be called, at a convenient time. */ |