gdb: un-share set_inferior_cwd declaration
authorSimon Marchi <simon.marchi@polymtl.ca>
Tue, 18 May 2021 15:27:43 +0000 (11:27 -0400)
committerSimon Marchi <simon.marchi@polymtl.ca>
Fri, 23 Jul 2021 19:38:54 +0000 (15:38 -0400)
The declaration of set_inferior_cwd is currently shared between gdb and
gdbserver, in gdbsupport/common-inferior.h.  It doesn't need to be, as
set_inferior_cwd is not called from common code.  Only get_inferior_cwd
needs to.

The motivation for this is that a future patch will change the prototype
of set_inferior_cwd in gdb, and I don't want to change it for gdbserver
unnecessarily.  I see this as a good cleanup in any case, to reduce to
just the essential what is shared between GDB and GDBserver.

Change-Id: I3127d27d078f0503ebf5ccc6fddf14f212426a73

gdb/infcmd.c
gdbserver/inferiors.cc
gdbserver/inferiors.h
gdbsupport/common-inferior.h

index 0a5edef698249fb17315de3b64e5b2a944ce7cc0..a7b520cdd16983fb044de72f569eeb872eb2c8a3 100644 (file)
@@ -184,9 +184,10 @@ show_args_command (struct ui_file *file, int from_tty,
   deprecated_show_value_hack (file, from_tty, c, get_inferior_args ());
 }
 
-/* See gdbsupport/common-inferior.h.  */
+/* Set the inferior current working directory.  If CWD is NULL, unset
+   the directory.  */
 
-void
+static void
 set_inferior_cwd (const char *cwd)
 {
   struct inferior *inf = current_inferior ();
index 3750090c0a14f95acfcc8f3a299b160a76b16766..0a09de79071d20c97352970c78ac85c844984c8b 100644 (file)
@@ -241,7 +241,7 @@ get_inferior_cwd ()
   return current_inferior_cwd;
 }
 
-/* See gdbsupport/common-inferior.h.  */
+/* See inferiors.h.  */
 
 void
 set_inferior_cwd (const char *cwd)
index 7754f745cd7d99e4387fc294b602d77b215f0d9b..3b8959a28cf7f2347fa85075860a0f61a3ab213b 100644 (file)
@@ -154,4 +154,8 @@ void *thread_target_data (struct thread_info *);
 struct regcache *thread_regcache_data (struct thread_info *);
 void set_thread_regcache_data (struct thread_info *, struct regcache *);
 
+/* Set the inferior current working directory.  If CWD is NULL, unset
+   the directory.  */
+void set_inferior_cwd (const char *cwd);
+
 #endif /* GDBSERVER_INFERIORS_H */
index 3f1b60fd96297cbdb0f4755fd3e138614598c0d2..5e1221277b873a73cfc58d68a2254378b9820d20 100644 (file)
@@ -36,10 +36,6 @@ extern const char *get_exec_file (int err);
    been set, then return NULL.  */
 extern const char *get_inferior_cwd ();
 
-/* Set the inferior current working directory.  If CWD is NULL, unset
-   the directory.  */
-extern void set_inferior_cwd (const char *cwd);
-
 /* Whether to start up the debuggee under a shell.
 
    If startup-with-shell is set, GDB's "run" will attempt to start up