This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[RFA v3 23/23] Remove make_cleanup_freeargv and gdb_buildargv
- From: Tom Tromey <tom at tromey dot com>
- To: gdb-patches at sourceware dot org
- Cc: Tom Tromey <tom at tromey dot com>
- Date: Wed, 2 Aug 2017 09:02:27 -0600
- Subject: [RFA v3 23/23] Remove make_cleanup_freeargv and gdb_buildargv
- Authentication-results: sourceware.org; auth=none
- References: <20170802150227.24460-1-tom@tromey.com>
After the previous patches in this series, make_cleanup_freeargv and
gdb_buildargv are now unused and can be removed.
ChangeLog
2017-08-02 Tom Tromey <tom@tromey.com>
* utils.c (make_cleanup_freeargv, do_freeargv, gdb_buildargv):
Remove.
* utils.h (make_cleanup_freeargv, gdb_buildargv): Remove.
---
gdb/ChangeLog | 6 ++++++
gdb/utils.c | 27 ---------------------------
gdb/utils.h | 4 ----
3 files changed, 6 insertions(+), 31 deletions(-)
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 259c37c..de8e046 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,11 @@
2017-08-02 Tom Tromey <tom@tromey.com>
+ * utils.c (make_cleanup_freeargv, do_freeargv, gdb_buildargv):
+ Remove.
+ * utils.h (make_cleanup_freeargv, gdb_buildargv): Remove.
+
+2017-08-02 Tom Tromey <tom@tromey.com>
+
* python/py-param.c (compute_enum_values): Use gdb_argv.
2017-08-02 Tom Tromey <tom@tromey.com>
diff --git a/gdb/utils.c b/gdb/utils.c
index e7e176d..96ae709 100644
--- a/gdb/utils.c
+++ b/gdb/utils.c
@@ -136,18 +136,6 @@ show_pagination_enabled (struct ui_file *file, int from_tty,
because while they use the "cleanup API" they are not part of the
"cleanup API". */
-static void
-do_freeargv (void *arg)
-{
- freeargv ((char **) arg);
-}
-
-struct cleanup *
-make_cleanup_freeargv (char **arg)
-{
- return make_cleanup (do_freeargv, arg);
-}
-
/* Helper function for make_cleanup_ui_out_redirect_pop. */
static void
@@ -2877,21 +2865,6 @@ gdb_argv::reset (const char *s)
m_argv = argv;
}
-/* Call libiberty's buildargv, and return the result.
- If buildargv fails due to out-of-memory, call nomem.
- Therefore, the returned value is guaranteed to be non-NULL,
- unless the parameter itself is NULL. */
-
-char **
-gdb_buildargv (const char *s)
-{
- char **argv = buildargv (s);
-
- if (s != NULL && argv == NULL)
- malloc_failure (0);
- return argv;
-}
-
int
compare_positive_ints (const void *ap, const void *bp)
{
diff --git a/gdb/utils.h b/gdb/utils.h
index 88cab4b..c922a22 100644
--- a/gdb/utils.h
+++ b/gdb/utils.h
@@ -85,8 +85,6 @@ extern int parse_pid_to_attach (const char *args);
extern int parse_escape (struct gdbarch *, const char **);
-char **gdb_buildargv (const char *);
-
/* A wrapper for an array of char* that was allocated in the way that
'buildargv' does, and should be freed with 'freeargv'. */
@@ -207,8 +205,6 @@ private:
/* Cleanup utilities. */
-extern struct cleanup *make_cleanup_freeargv (char **);
-
struct ui_out;
extern struct cleanup *
make_cleanup_ui_out_redirect_pop (struct ui_out *uiout);
--
2.9.4