Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Stan Shebs <stanshebs@earthlink.net>
To: Andreas Schwab <schwab@linux-m68k.org>
Cc: gdb-patches@sourceware.org
Subject: Re: [PATCH] dynamic printf
Date: Tue, 13 Mar 2012 23:09:00 -0000	[thread overview]
Message-ID: <4F5FD3A6.1090106@earthlink.net> (raw)
In-Reply-To: <m2fwduc883.fsf@igel.home>

On 2/29/12 1:14 AM, Andreas Schwab wrote:
> Stan Shebs<stanshebs@earthlink.net>  writes:
>
>> This patch implements a "dynamic printf", which is basically a breakpoint
>> with a printf;continue as its command list - but with additional features
>> that make it more interesting.
> How about:
>
> break [LOCATION] [thread THREADNUM] [if CONDITION] [printf FORMAT,ARGS...]
>
> Andreas.
>

I've been wondering whether that might be a better idea.  An advantage 
of "dprintf" for the command line is that it can abbreviate to just "dp" 
and tab completion works, while the optional arguments to the break 
command don't have any way to abbreviate (although they probably 
could).  That's not super-compelling, as by its nature a printf command 
entails typing in format string, double quotes and newlines and all, and 
in practice I expect a lot of these will tend to accumulate in script files.

Another consideration is how the collection looks in an info break 
command.  If dynamic prints are a different kind of breakpoint, then you 
can (in theory) filter the breakpoint listing to a single category, but 
if they are just breakpoints with special commands, they'll continue to 
be in the big list.

My intuition isn't giving me a strong read on which approach users will 
like better.  As the goal is at least partly to entice 
printf()+recompile users into using a GDB command instead, I think it 
needs to be among GDB's simpler commands.  Maybe include both syntaxes, 
mapping down to same functionality?

Stan



  reply	other threads:[~2012-03-13 23:09 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-29  9:14 Stan Shebs
2012-02-29 10:28 ` Andreas Schwab
2012-03-13 23:09   ` Stan Shebs [this message]
2012-03-14 14:47     ` Marc Khouzam
2012-03-14 15:28     ` Tom Tromey
2012-02-29 16:16 ` Joel Brobecker
2012-03-13 23:15   ` Stan Shebs
2012-03-13 23:40     ` Joel Brobecker
2012-02-29 18:21 ` Eli Zaretskii
2012-03-13 23:20   ` Stan Shebs
2012-03-01 14:04 ` Hui Zhu
2012-03-04  6:31   ` Hui Zhu
2012-03-08 21:08 ` Tom Tromey
2012-03-13 23:51   ` Stan Shebs
2012-03-14 15:24     ` Tom Tromey

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4F5FD3A6.1090106@earthlink.net \
    --to=stanshebs@earthlink.net \
    --cc=gdb-patches@sourceware.org \
    --cc=schwab@linux-m68k.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox