From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 7111 invoked by alias); 9 Mar 2004 00:12:20 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 7061 invoked from network); 9 Mar 2004 00:12:17 -0000 Received: from unknown (HELO localhost.redhat.com) (216.129.200.20) by sources.redhat.com with SMTP; 9 Mar 2004 00:12:17 -0000 Received: from gnu.org (localhost [127.0.0.1]) by localhost.redhat.com (Postfix) with ESMTP id 9BF622B92; Mon, 8 Mar 2004 19:12:17 -0500 (EST) Message-ID: <404D0BE1.90803@gnu.org> Date: Tue, 09 Mar 2004 00:12:00 -0000 From: Andrew Cagney User-Agent: Mozilla/5.0 (X11; U; NetBSD macppc; en-GB; rv:1.4.1) Gecko/20040217 MIME-Version: 1.0 To: Eli Zaretskii , Daniel Jacobowitz Cc: gdb-patches@sources.redhat.com Subject: Re: [rfa/target/doc] Print less with "set debug target 1" References: <20040306230834.GA9343@nevyn.them.org> <3791-Sun07Mar2004073738+0200-eliz@elta.co.il> In-Reply-To: <3791-Sun07Mar2004073738+0200-eliz@elta.co.il> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2004-03.o/txt/msg00186.txt >>Date: Sat, 6 Mar 2004 18:08:35 -0500 >>> From: Daniel Jacobowitz >>> >>> Seem OK? > > > Thanks for the documentation patch, but I have 2 minor comments: > > >>> @item set debug target >>> Turns on or off display of @value{GDBN} target debugging info. This info >>> includes what is going on at the target level of GDB, as it happens. The >>> -default is off. >>> +default is 0. Set it to 1 to track events, and to 2 to also track the >>> +value of large memory transfers. This flag must be set before connecting >>> +to the target or saying @code{run}. > > > . I'm not sure I understand this sentence--do you mean to say that > _if_ the flag is set, it must be set _before_ connecting and > before "run"? If so, I suggest to use @emph and/or @strong to > make sure the reader understands you, and maybe rephrase slightly > to make sure the wording is less ambiguous. > > . Since in this context "run" is something the user types, it > should be @kbd{run}, not @code{run}. Bad design. This doesn't work: (gdb) set debug target 1 ... do something, get target debug output ... (gdb) set debug target 0 ... do something, no target debug output ... No technical reason for the behavior (I suspect it is that way 'cos the target vector is still macros and 'cos people thought that always using an extra level of indirection would make things slow). Daniel, just also file a bug report, and feel free to add some extra verbage to: add_show_from_set (add_set_cmd ("target", class_maintenance, var_zinteger, (char *) &targetdebug, "Set target debugging.\n\ When non-zero, target debugging is enabled.", &setdebuglist), &showdebuglist); Andrew > Otherwise, it is okay to go in. > From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 7111 invoked by alias); 9 Mar 2004 00:12:20 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 7061 invoked from network); 9 Mar 2004 00:12:17 -0000 Received: from unknown (HELO localhost.redhat.com) (216.129.200.20) by sources.redhat.com with SMTP; 9 Mar 2004 00:12:17 -0000 Received: from gnu.org (localhost [127.0.0.1]) by localhost.redhat.com (Postfix) with ESMTP id 9BF622B92; Mon, 8 Mar 2004 19:12:17 -0500 (EST) Message-ID: <404D0BE1.90803@gnu.org> Date: Fri, 19 Mar 2004 00:09:00 -0000 From: Andrew Cagney User-Agent: Mozilla/5.0 (X11; U; NetBSD macppc; en-GB; rv:1.4.1) Gecko/20040217 MIME-Version: 1.0 To: Eli Zaretskii , Daniel Jacobowitz Cc: gdb-patches@sources.redhat.com Subject: Re: [rfa/target/doc] Print less with "set debug target 1" References: <20040306230834.GA9343@nevyn.them.org> <3791-Sun07Mar2004073738+0200-eliz@elta.co.il> In-Reply-To: <3791-Sun07Mar2004073738+0200-eliz@elta.co.il> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2004-03/txt/msg00186.txt.bz2 Message-ID: <20040319000900.X1uoNyfZj_umkNqntXfjWE9HXVJZPQLUakn50PWaKdM@z> >>Date: Sat, 6 Mar 2004 18:08:35 -0500 >>> From: Daniel Jacobowitz >>> >>> Seem OK? > > > Thanks for the documentation patch, but I have 2 minor comments: > > >>> @item set debug target >>> Turns on or off display of @value{GDBN} target debugging info. This info >>> includes what is going on at the target level of GDB, as it happens. The >>> -default is off. >>> +default is 0. Set it to 1 to track events, and to 2 to also track the >>> +value of large memory transfers. This flag must be set before connecting >>> +to the target or saying @code{run}. > > > . I'm not sure I understand this sentence--do you mean to say that > _if_ the flag is set, it must be set _before_ connecting and > before "run"? If so, I suggest to use @emph and/or @strong to > make sure the reader understands you, and maybe rephrase slightly > to make sure the wording is less ambiguous. > > . Since in this context "run" is something the user types, it > should be @kbd{run}, not @code{run}. Bad design. This doesn't work: (gdb) set debug target 1 ... do something, get target debug output ... (gdb) set debug target 0 ... do something, no target debug output ... No technical reason for the behavior (I suspect it is that way 'cos the target vector is still macros and 'cos people thought that always using an extra level of indirection would make things slow). Daniel, just also file a bug report, and feel free to add some extra verbage to: add_show_from_set (add_set_cmd ("target", class_maintenance, var_zinteger, (char *) &targetdebug, "Set target debugging.\n\ When non-zero, target debugging is enabled.", &setdebuglist), &showdebuglist); Andrew > Otherwise, it is okay to go in. >