Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Keith Seitz <keiths@cygnus.com>
To: Eli Zaretskii <eliz@is.elta.co.il>
Cc: <fnasser@redhat.com>, <gdb-patches@sources.redhat.com>
Subject: Re: [RFA] breakpoint.c: don't generate bp events for internal bps
Date: Fri, 11 May 2001 08:14:00 -0000	[thread overview]
Message-ID: <Pine.GSO.4.33.0105110805480.27065-100000@ryobi.cygnus.com> (raw)
In-Reply-To: <3099-Fri11May2001180501+0300-eliz@is.elta.co.il>

On Fri, 11 May 2001, Eli Zaretskii wrote:

> > Do the words "it's stupid" mean anything to you?
>
> I looked them up ;-)

Touche!

> > From gdb-events.h:
> >
> > /* User Interface Events.
> >    Copyright 1999, 2000, 2001 Free Software Foundation, Inc.
> >
> > "User Interface Events". User interfaces don't care about internal
> > breakpoints -- only user-set breakpoints. IMO all of the breakpoints
> > excluded from the event trigger are BACKEND breakpoints, and they should
> > not ever be exposed to users (or non-backend code).
> >
> > If someone writing ui code needs to know about internal breakpoints, he
> > doing something wrong.
>
> Sorry, I still don't get the relevance of this to what I asked.
> Perhaps I'm stupid.

No, I'm not explaining myself properly... Let me try again:

breakpoint_delete_event, breakpoint_create_event, and
breakpoint_modify_event are defined in gdb-events.[ch],sh. They are, as
the comments from the files explain, for user interface events.

Well, it is my contention that user interfaces need not (indeed, should
not) know anything about bp_none, bp_until, bp_finish, bp_longjmp,
bp_longjmp_resume, bp_step_resume, bp_through_sigtramp,
bp_watchpoint_scope, bp_call_dummy, bp_shlibevent, bp_thread_event,
bp_catch_unload, bp_catch_fork, bp_catch_vfork, bp_catch_exec,
bp_catch_catch, and bp_catch_throw breakpoint events (or some major subset
of these). What libgdb does for its own housekeeping is its business, and
its business alone. Breakpoints of these types are PRIVATE to gdb.

(I suspect this isn't helping too much, is it?)

I would even go further to argue with Fernando that we need to eliminate
modify_breakpoint_hook, delete_breakpoint_hook, and
create_breakpoint_hook, because these are not hooks: they are events. We
already have an event-reporting mechanism. Let people use it. :-)

I'm in the mood for making friends today! O:-)
Keith



  reply	other threads:[~2001-05-11  8:14 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <Pine.GSO.4.33.0105102023220.29679-100000@ryobi.cygnus.com>
2001-05-10 22:26 ` Fernando Nasser
2001-05-10 23:56   ` Eli Zaretskii
2001-05-11  7:09     ` Keith Seitz
2001-05-11  8:04       ` Eli Zaretskii
2001-05-11  8:14         ` Keith Seitz [this message]
2001-05-11  9:57           ` Eli Zaretskii
2001-05-11  7:05   ` Keith Seitz
2001-05-11  8:54     ` Fernando Nasser
2001-05-11 18:56 ` Andrew Cagney
     [not found] <Pine.GSO.4.33.0105110855100.9875-100000@ryobi.cygnus.com>
2001-05-11  9:15 ` Fernando Nasser
2001-05-11 10:22   ` Michael Snyder
2001-05-11 10:25     ` Fernando Nasser

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.GSO.4.33.0105110805480.27065-100000@ryobi.cygnus.com \
    --to=keiths@cygnus.com \
    --cc=eliz@is.elta.co.il \
    --cc=fnasser@redhat.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