Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Hui Zhu <teawater@gmail.com>
To: tromey@redhat.com
Cc: Marc Khouzam <marc.khouzam@ericsson.com>,
	Pedro Alves <pedro@codesourcery.com>,
	 	"gdb-patches@sourceware.org" <gdb-patches@sourceware.org>,
	Joel Brobecker <brobecker@adacore.com>,
	 	Michael Snyder <msnyder@vmware.com>
Subject: Re: Another proposal for frontends and queries.
Date: Wed, 16 Sep 2009 23:36:00 -0000	[thread overview]
Message-ID: <daef60380909161635u6960123fw67031a333f54f486@mail.gmail.com> (raw)
In-Reply-To: <m3ab0urcnc.fsf@fleche.redhat.com>

On Thu, Sep 17, 2009 at 02:02, Tom Tromey <tromey@redhat.com> wrote:
>>>>>> ">" == Hui Zhu <teawater@gmail.com> writes:
>
>>> And about the patch I said can handle this issue is "set record query
>>> <on|off>" patch.  I cannot find who don't like this patch.
>
> I don't like it, I think I said that in another thread.
>
> The reason I don't like it is twofold.
>
> First, queries are a general feature of gdb.  They are sprinkled about
> rather liberally.  So, if one causes problems for a front end, then it
> probably represents a general class of problems.  IMO, fixing it in this
> ad hoc way is not a good way to attack a generic problem.
>
> The thought experiment to perform here is to consider generalizing this
> solution to all the queries in gdb.  I think that would result in
> madness: hundreds of "disable this query" settings.
>
> Second, this has the feeling of an "unbreak my gdb" option.  That is,
> setting an option to avoid one particular query says to me that the
> query is probably badly chosen in some way.
>

Sorry I didn't get the mean of "It seems like a strange approach to
me." first time.  Now, I got it.

One day, I try to debug a big program with GDB prec. I don't know why
it get something wrong.  It get bug only after exec a long time.  And
prec will make inferior exec slow (I design dump and skip to make it
fast).  So I enter "record" and "c" command in GDB and go to bed.
In the next morning, I open the LCD and want see what happen.  I found
that GDB is broken by a query.  :(

That is why I like "set rec query off".  It will not be a only switch.
 It will include a lot of switches for each query.  For the advanced
user, he can set each switch and then set rec query off.  Then he can
let GDB unbreak record a long time, and come back to use replay debug
find what happen.

That is why I think "set prec query off" is not bad.

Thanks,
Hui


  reply	other threads:[~2009-09-16 23:36 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-14  2:38 Marc Khouzam
2009-09-14  6:45 ` Vladimir Prus
2009-09-14 13:07   ` Marc Khouzam
2009-09-14 14:09 ` Joel Brobecker
2009-09-14 14:28   ` Marc Khouzam
2009-09-14 14:48     ` Pedro Alves
2009-09-14 14:57       ` Marc Khouzam
2009-09-14 15:08       ` Joel Brobecker
2009-09-14 15:36         ` Marc Khouzam
2009-09-14 16:06           ` Joel Brobecker
2009-09-15 16:26         ` Joel Brobecker
2009-09-16  6:24           ` Hui Zhu
2009-09-16 12:38             ` Marc Khouzam
2009-09-16 13:06               ` Hui Zhu
2009-09-16 13:25                 ` Pedro Alves
2009-09-16 13:46                   ` Hui Zhu
2009-09-16 13:59                     ` Marc Khouzam
2009-09-16 14:17                       ` Hui Zhu
2009-09-16 14:26                         ` Joel Brobecker
2009-09-16 14:34                           ` Hui Zhu
2009-09-16 14:45                             ` Pedro Alves
2009-09-16 14:43                           ` Marc Khouzam
2009-09-16 18:03                         ` Tom Tromey
2009-09-16 23:36                           ` Hui Zhu [this message]
2009-09-16 23:40                             ` Joel Brobecker
2009-09-16 17:12                   ` Eli Zaretskii
2009-09-14 15:23     ` Joel Brobecker
2009-09-16 20:20     ` Tom Tromey
2009-09-16 20:35       ` Marc Khouzam
2009-09-17  0:17         ` Michael Snyder
2009-09-17  0:48           ` Marc Khouzam
2009-09-17  1:03             ` Joel Brobecker
2009-09-17  2:07               ` Marc Khouzam
2009-09-17  0:16       ` Michael Snyder
2009-09-16 20:43 ` Tom Tromey
2009-09-16 21:41   ` Joel Brobecker
2009-09-17 13:51     ` Marc Khouzam
2009-09-17 19:52       ` Tom Tromey
2009-09-18  4:59         ` Hui Zhu
2009-09-18  5:16           ` Michael Snyder
2009-09-18  9:31           ` Pedro Alves
2009-09-18 14:37             ` Joel Brobecker
2009-09-18 14:57         ` Marc Khouzam
2009-09-18 16:42           ` Tom Tromey
2009-09-22  0:46             ` Marc Khouzam
2009-09-22 22:49               ` [RFA/commit] s/nquery/query/ in record.c (was "Re: Another proposal for frontends and queries.") Joel Brobecker
2009-09-23 12:45                 ` Marc Khouzam
2009-09-24  3:05                   ` Hui Zhu
2009-09-24 17:59                 ` Joel Brobecker

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=daef60380909161635u6960123fw67031a333f54f486@mail.gmail.com \
    --to=teawater@gmail.com \
    --cc=brobecker@adacore.com \
    --cc=gdb-patches@sourceware.org \
    --cc=marc.khouzam@ericsson.com \
    --cc=msnyder@vmware.com \
    --cc=pedro@codesourcery.com \
    --cc=tromey@redhat.com \
    /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