From: Eli Zaretskii <eliz@is.elta.co.il>
To: Andrew Cagney <ac131313@cygnus.com>
Cc: gdb-patches@sources.redhat.com
Subject: Re: [RFA] Make access watchpoints work again
Date: Tue, 20 Mar 2001 01:54:00 -0000 [thread overview]
Message-ID: <Pine.SUN.3.91.1010320115153.7756a-100000@is> (raw)
In-Reply-To: <3AB63A0B.FB05689E@cygnus.com>
On Mon, 19 Mar 2001, Andrew Cagney wrote:
> > (Btw, why isn't the case, where an access watchpoint triggers, but the
> > value didn't change, being annotated with a call to
> > annotate_watchpoint?)
>
> If it wasn't being done in the pre- ui-out code, I'd assume it's because
> no one noticed it missing :-) If it was being done then I'd assume a bug
> in the conversion to ui-out :-(
No, the conversion to ui-out seems to properly follow what's been
there before: any watchpoint which triggers is not annotated if the
watched expression's value is unchanged.
The only way I can understand this is that the users of the annotation
interface did something which doesn't make sense when the value hasn't
changed. I don't have experience with any application that uses
annotations, so I cannot figure out whether this hypothesis is true or
not. Anybody?
FWIW, annotate.texi doesn't say anything about read watchpoints being
unannotated.
If no one can find the reason, how about adding the missing annotation
and waiting for someone to scream?
> PS: Assume the ui-out change is an obvious fix.
Commited.
next prev parent reply other threads:[~2001-03-20 1:54 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <200103032150.QAA29021@indy.delorie.com>
2001-03-07 1:20 ` [RFA] Unified watchpoints for x86 platforms Eli Zaretskii
2001-03-07 7:56 ` Mark Kettenis
2001-03-07 9:23 ` Eli Zaretskii
2001-03-09 14:05 ` Mark Kettenis
2001-03-11 3:19 ` Eli Zaretskii
2001-03-14 5:11 ` Mark Kettenis
2001-03-17 9:18 ` Eli Zaretskii
2001-03-17 14:54 ` Mark Kettenis
2001-03-18 0:57 ` Eli Zaretskii
2001-03-17 15:20 ` Mark Kettenis
2001-03-18 0:58 ` Eli Zaretskii
2001-03-18 12:47 ` [RFA] Make access watchpoints work again Eli Zaretskii
2001-03-19 8:56 ` Andrew Cagney
2001-03-20 1:54 ` Eli Zaretskii [this message]
2001-03-23 8:06 ` Andrew Cagney
2001-03-17 9:21 ` [RFA] Unified watchpoints for x86 platforms Eli Zaretskii
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=Pine.SUN.3.91.1010320115153.7756a-100000@is \
--to=eliz@is.elta.co.il \
--cc=ac131313@cygnus.com \
--cc=gdb-patches@sources.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