From patchwork Wed Mar 12 12:37:52 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Pedro Alves X-Patchwork-Id: 49 Return-Path: X-Original-To: siddhesh@wilcox.dreamhost.com Delivered-To: siddhesh@wilcox.dreamhost.com Received: from homiemail-mx21.g.dreamhost.com (caibbdcaaahb.dreamhost.com [208.113.200.71]) by wilcox.dreamhost.com (Postfix) with ESMTP id 7554C36008F for ; Wed, 12 Mar 2014 05:38:04 -0700 (PDT) Received: by homiemail-mx21.g.dreamhost.com (Postfix, from userid 14314964) id 295E11B10C33; Wed, 12 Mar 2014 05:38:04 -0700 (PDT) X-Original-To: gdb@patchwork.siddhesh.in Delivered-To: x14314964@homiemail-mx21.g.dreamhost.com Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by homiemail-mx21.g.dreamhost.com (Postfix) with ESMTPS id ECB461A69A86 for ; Wed, 12 Mar 2014 05:38:03 -0700 (PDT) DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:from:to:subject:date:message-id; q=dns; s= default; b=JDZKJyrmX8NdGHqmE6NPIMduAdd+XpM9If39faJa7n2hcu+7j6WDt LZO9Qp4gVzcgWfD7MgrExCng9FhYYiEP+rRQG8WjFCxtpeVmbQ4w44fqxxGVgz3n ZabAfw4qsI4LKPRZNdudE51LKjbqoIm20zSuTh5zUSW80B9mr/FNsQ= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:from:to:subject:date:message-id; s=default; bh=+EdqcklJZuurTC4BJDIoHcTCgbM=; b=f49r01gv73Ha763nPWtT868Kizkn 0lHPaEom4iyXprLQeBIUMe9Yd27dD0bWe8EQkKZSUptrECwmDRJ9e74y3WQPl/65 Ptcrjr/3Oka3z7oqUvP57R55/rNK8Dwp7jjB0Ja+WvPrfTLJJEaDkjeBRPucizm9 TJcjh799eHX5+WU= Received: (qmail 22702 invoked by alias); 12 Mar 2014 12:38:02 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 22690 invoked by uid 89); 12 Mar 2014 12:38:02 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.8 required=5.0 tests=AWL, BAYES_00, SPF_HELO_PASS, SPF_PASS, T_RP_MATCHES_RCVD autolearn=ham version=3.3.2 X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 12 Mar 2014 12:38:01 +0000 Received: from int-mx01.intmail.prod.int.phx2.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id s2CCbrbm026339 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Wed, 12 Mar 2014 08:37:54 -0400 Received: from brno.lan (ovpn01.gateway.prod.ext.ams2.redhat.com [10.39.146.11]) by int-mx01.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id s2CCbqje014243 for ; Wed, 12 Mar 2014 08:37:53 -0400 From: Pedro Alves To: gdb-patches@sourceware.org Subject: [PATCH] Don't mention "Unix" in native target name. Date: Wed, 12 Mar 2014 12:37:52 +0000 Message-Id: <1394627872-18619-1-git-send-email-palves@redhat.com> X-DH-Original-To: gdb@patchwork.siddhesh.in 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". gdb/ 2014-03-12 Pedro Alves * inf-child.c (inf_child_open, inf_child_target): Don't mention Unix in user visible strings. --- gdb/inf-child.c | 6 +++--- gdb/testsuite/gdb.base/default.exp | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/gdb/inf-child.c b/gdb/inf-child.c index ee63dd1..bfa9152 100644 --- a/gdb/inf-child.c +++ b/gdb/inf-child.c @@ -112,7 +112,7 @@ inf_child_prepare_to_store (struct target_ops *self, static void inf_child_open (char *arg, int from_tty) { - error (_("Use the \"run\" command to start a Unix child process.")); + error (_("Use the \"run\" command to start a child process.")); } static void @@ -398,8 +398,8 @@ inf_child_target (void) struct target_ops *t = XCNEW (struct target_ops); t->to_shortname = "child"; - t->to_longname = "Unix child process"; - t->to_doc = "Unix child process (started by the \"run\" command)."; + t->to_longname = "Child process"; + t->to_doc = "Child process (started by the \"run\" command)."; t->to_open = inf_child_open; t->to_post_attach = inf_child_post_attach; t->to_fetch_registers = inf_child_fetch_inferior_registers; diff --git a/gdb/testsuite/gdb.base/default.exp b/gdb/testsuite/gdb.base/default.exp index 73c33ba..b63955e 100644 --- a/gdb/testsuite/gdb.base/default.exp +++ b/gdb/testsuite/gdb.base/default.exp @@ -747,10 +747,10 @@ gdb_test "step" "The program is not being run." "step #2" gdb_test "symbol-file" ".*" "symbol-file" #test target child -gdb_test "target child" "Use the \"run\" command to start a Unix child process.*|Undefined target command: \"child\". *Try \"help target\".*" "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 Unix child process.*|Undefined target command: \"procfs\". *Try \"help target\".*" "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"