This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Rename Solaris's target to "target child" like most other ports.


On 03/12/2014 12:37 PM, Pedro Alves wrote:
> I wonder whether anyone would miss this badly?  The difference would be:
> 
>   (gdb) maint print target-stack
>   The current target stack is:
>    - multi-thread (multi-threaded child process.)
> -  - child (Unix child process)
> +  - child (Child process)
>    - exec (Local exec file)
>    - None (None)
> 
>  (gdb) help target child
> - Unix child process (started by the "run" command).
> + Child process (started by the "run" command).
> 
> I find it unnecessary (and really slightly a lie) on GNU/Linux.
> 
> It's also odd that e.g., the Windows port says "Unix" in reaction to
> "target child" (it was already that way before Windows used
> inf-child.c).
> 
> How about we just say the same mostly everywhere?
> 
> In the same vein, I'd also rename Solaris/procfs.c's "target procfs"
> to "target child".

Like this.  (Untested.)

-------
[PATCH] Rename Solaris's target to "target child" like most other
 ports.

Note that "target procfs" is used by QNX, but the test must be failing
there, as nto-procfs.c overrides to_open with a method that doesn't
throw the error being tested.  So I'm just removing the test
completely.

gdb/
2014-03-12  Pedro Alves  <palves@redhat.com>

	* procfs.c (procfs_target): Don't override to_shortname,
	to_longname or to_doc.

gdb/testsuite/
2014-03-12  Pedro Alves  <palves@redhat.com>

	* gdb.base/default.exp: Don't test "target procfs".
---
 gdb/procfs.c                       | 4 ----
 gdb/testsuite/gdb.base/default.exp | 3 ---
 2 files changed, 7 deletions(-)

diff --git a/gdb/procfs.c b/gdb/procfs.c
index 8204747..f0d65d2 100644
--- a/gdb/procfs.c
+++ b/gdb/procfs.c
@@ -184,10 +184,6 @@ procfs_target (void)
 {
   struct target_ops *t = inf_child_target ();
 
-  t->to_shortname = "procfs";
-  t->to_longname = "Unix /proc child process";
-  t->to_doc =
-    "Unix /proc child process (started by the \"run\" command).";
   t->to_create_inferior = procfs_create_inferior;
   t->to_kill = procfs_kill_inferior;
   t->to_mourn_inferior = procfs_mourn_inferior;
diff --git a/gdb/testsuite/gdb.base/default.exp b/gdb/testsuite/gdb.base/default.exp
index b63955e..a39ffcf 100644
--- a/gdb/testsuite/gdb.base/default.exp
+++ b/gdb/testsuite/gdb.base/default.exp
@@ -749,9 +749,6 @@ gdb_test "symbol-file" ".*" "symbol-file"
 #test target child
 gdb_test "target child" "Use the \"run\" command to start a child process.*|Undefined target command: \"child\".  *Try \"help target\".*" "target child"
 
-#test target procfs
-gdb_test "target procfs" "Use the \"run\" command to start a child process.*|Undefined target command: \"procfs\".  *Try \"help target\".*" "target procfs"
-
 #test target core
 send_gdb "target core\n"
 gdb_expect {
-- 
1.7.11.7



-- 
Pedro Alves


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]