From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-99997-listarch-gdb-patches=sources.redhat.com@sourceware.org>
Received: (qmail 17634 invoked by alias); 28 Mar 2013 14:38:44 -0000
Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Id: <gdb-patches.sourceware.org>
List-Subscribe: <mailto:gdb-patches-subscribe@sourceware.org>
List-Archive: <http://sourceware.org/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sourceware.org>
List-Help: <mailto:gdb-patches-help@sourceware.org>, <http://sourceware.org/ml/#faqs>
Sender: gdb-patches-owner@sourceware.org
Received: (qmail 17607 invoked by uid 89); 28 Mar 2013 14:38:37 -0000
X-Spam-SWARE-Status: No, score=-4.4 required=5.0 tests=AWL,BAYES_00,KHOP_THREADED,RCVD_IN_DNSWL_NONE,RCVD_IN_HOSTKARMA_NO,RCVD_IN_HOSTKARMA_YE,SPF_SOFTFAIL autolearn=no version=3.3.1
Received: from mtaout22.012.net.il (HELO mtaout22.012.net.il) (80.179.55.172)    by sourceware.org (qpsmtpd/0.84/v0.84-167-ge50287c) with ESMTP; Thu, 28 Mar 2013 14:38:33 +0000
Received: from conversion-daemon.a-mtaout22.012.net.il by a-mtaout22.012.net.il (HyperSendmail v2007.08) id <0MKD00G00KLEBY00@a-mtaout22.012.net.il> for gdb-patches@sourceware.org; Thu, 28 Mar 2013 16:38:30 +0200 (IST)
Received: from HOME-C4E4A596F7 ([87.69.4.28]) by a-mtaout22.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0MKD00FDBKO6E5K0@a-mtaout22.012.net.il>; Thu, 28 Mar 2013 16:38:30 +0200 (IST)
Date: Thu, 28 Mar 2013 18:37:00 -0000
From: Eli Zaretskii <eliz@gnu.org>
Subject: Re: [PATCH] add -s option to make -break-insert support dprintf
In-reply-to: <515451EA.1000200@mentor.com>
To: Hui Zhu <hui_zhu@mentor.com>
Cc: gdb-patches@sourceware.org, marc.khouzam@ericsson.com
Reply-to: Eli Zaretskii <eliz@gnu.org>
Message-id: <83y5d7wpvq.fsf@gnu.org>
References: <515451EA.1000200@mentor.com>
X-SW-Source: 2013-03/txt/msg01062.txt.bz2

> Date: Thu, 28 Mar 2013 22:21:30 +0800
> From: Hui Zhu <hui_zhu@mentor.com>
> CC: Marc Khouzam <marc.khouzam@ericsson.com>, Eli Zaretskii <eliz@gnu.org>
> 
> This is the patches to add -s flags to make -break-info support dprintf.
> mi-dprintf.txt is for the code.
> mi-dprintf-doc.txt is for the doc.
> mi-dprintf-test.txt is for the test.
> 
> After this patch, you can use -s option to add dprintf.  To see mi-dprintf-doc.txt to get info about how to use this function.

Thanks.

> This is for NEWS:
>    ** The -s of MI command -break-insert can set a dynamic printf.

 "The new option -s of the MI command -break-insert sets a dynamic
  printf breakpoint."

> 2013-03-28  Hui Zhu  <hui@codesourcery.com>
> 
> 	* gdb.texinfo (GDB/MI Breakpoint Commands): Add "-s".
                                                    ^^^^^^^^
"Describe the -s option."

> --- a/doc/gdb.texinfo
> +++ b/doc/gdb.texinfo
> @@ -28784,7 +28784,9 @@ N.A.
>  @smallexample
>   -break-insert [ -t ] [ -h ] [ -f ] [ -d ] [ -a ]
>      [ -c @var{condition} ] [ -i @var{ignore-count} ]
> -    [ -p @var{thread-id} ] [ @var{location} ]
> +    [ -p @var{thread-id} ]
> +    [ -s "@var{template},@var{expression}[,@var{expression}@dots{}]" ]
> +    [ @var{location} ]
>  @end smallexample
>  
>  @noindent
> @@ -28824,6 +28826,29 @@ Make the breakpoint conditional on @var{
>  Initialize the @var{ignore-count}.
>  @item -p @var{thread-id}
>  Restrict the breakpoint to the specified @var{thread-id}.
> +@item -s "@var{location},@var{template},@var{expression}[,@var{expression}@dots{}]"

The format of the -s argument in @item is different from the format in
the @smallexample above.  Which one is correct?

> +Set a @ref{Dynamic Printf}.

Please don't use @ref as if it were an HTML "<a href" attribute; it is
not.  This produces ugly Info output.  Please use this style instead:

  Set a dynamic printf breakpoint (@pxref{Dynamic Printf}).

or this:

  Set a dynamic printf breakpoint, described in @ref{Dynamic Printf}.