This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[RFA 42/67] Constify some commands in exec.c, plus symbol_file_command
- 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, 20 Sep 2017 23:09:58 -0600
- Subject: [RFA 42/67] Constify some commands in exec.c, plus symbol_file_command
- Authentication-results: sourceware.org; auth=none
- References: <20170921051023.19023-1-tom@tromey.com>
Note that this commit also changes deprecated_file_changed_hook -- not
used in the tree, but Insight will require a (presumably minor)
change.
ChangeLog
2017-09-20 Tom Tromey <tom@tromey.com>
* symfile.c (symbol_file_command): Constify.
* gdbcore.h (deprecated_file_changed_hook): Constify.
* exec.c (deprecated_file_changed_hook, exec_file_command)
(file_command): Constify.
* defs.h (symbol_file_command): Constify.
---
gdb/ChangeLog | 8 ++++++++
gdb/defs.h | 2 +-
gdb/exec.c | 8 +++-----
gdb/gdbcore.h | 2 +-
gdb/symfile.c | 2 +-
5 files changed, 14 insertions(+), 8 deletions(-)
diff --git a/gdb/defs.h b/gdb/defs.h
index af9e32e..a25ebf6 100644
--- a/gdb/defs.h
+++ b/gdb/defs.h
@@ -304,7 +304,7 @@ EXTERN_C char *re_comp (const char *);
/* From symfile.c */
-extern void symbol_file_command (char *, int);
+extern void symbol_file_command (const char *, int);
/* * Remote targets may wish to use this as their load function. */
extern void generic_load (const char *name, int from_tty);
diff --git a/gdb/exec.c b/gdb/exec.c
index d20afdc..6eda9b2 100644
--- a/gdb/exec.c
+++ b/gdb/exec.c
@@ -45,12 +45,10 @@
#include "solist.h"
#include <algorithm>
-void (*deprecated_file_changed_hook) (char *);
+void (*deprecated_file_changed_hook) (const char *);
/* Prototypes for local functions */
-static void file_command (char *, int);
-
static void set_section_command (char *, int);
static void exec_files_info (struct target_ops *);
@@ -398,7 +396,7 @@ exec_file_attach (const char *filename, int from_tty)
If ARGS is NULL, we just want to close the exec file. */
static void
-exec_file_command (char *args, int from_tty)
+exec_file_command (const char *args, int from_tty)
{
char *filename;
@@ -433,7 +431,7 @@ exec_file_command (char *args, int from_tty)
command was added? */
static void
-file_command (char *arg, int from_tty)
+file_command (const char *arg, int from_tty)
{
/* FIXME, if we lose on reading the symbol file, we should revert
the exec file, but that's rough. */
diff --git a/gdb/gdbcore.h b/gdb/gdbcore.h
index 7554245..5c86e98 100644
--- a/gdb/gdbcore.h
+++ b/gdb/gdbcore.h
@@ -127,7 +127,7 @@ extern void (*deprecated_exec_file_display_hook) (const char *filename);
/* Hook for "file_command", which is more useful than above
(because it is invoked AFTER symbols are read, not before). */
-extern void (*deprecated_file_changed_hook) (char *filename);
+extern void (*deprecated_file_changed_hook) (const char *filename);
extern void specify_exec_file_hook (void (*hook) (const char *filename));
diff --git a/gdb/symfile.c b/gdb/symfile.c
index c6e3b90..c6d657b 100644
--- a/gdb/symfile.c
+++ b/gdb/symfile.c
@@ -1617,7 +1617,7 @@ find_separate_debug_file_by_debuglink (struct objfile *objfile)
conventions (because it is confusing and inconvenient). */
void
-symbol_file_command (char *args, int from_tty)
+symbol_file_command (const char *args, int from_tty)
{
dont_repeat ();
--
2.9.4