public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
From: Doug Evans <dje@google.com>
To: Kevin Pouget <kevin.pouget@gmail.com>
Cc: gdb-patches@sourceware.org, pmuldoon@redhat.com,
		Eli Zaretskii <eliz@gnu.org>
Subject: Re: [PATCH] Add bp_location to Python interface
Date: Wed, 11 Jan 2012 19:45:00 -0000	[thread overview]
Message-ID: <CADPb22RTZmwQ3VsrYYw--o=qUp+3_FOT6Sx6BU5dkcP9uGKmkw@mail.gmail.com> (raw)
In-Reply-To: <CAPftXU+R6kZqwF0ZFhc_dBQW54z__581rgLgbift3DcGLcCLhA@mail.gmail.com>

On Jan 11, 2012 1:02 AM, "Kevin Pouget" <kevin.pouget@gmail.com> wrote:
>
> I'm not sure we can really move it away from the Python/Guile API,
> because, AFAIU, this volatile aspect if purely internal. 'Normal'
> users won't ever see it.
>
> > $ i b
> > Num     Type           Disp Enb Address            What
> > 1       breakpoint     keep y    <MULTIPLE>
> >         1.1                         y     0x4562c0 in main at ../../../git/gdb/gdb/gdb.c:26 inf 2
> >         1.2                         y     0x4562c0 in main at ../../../git/gdb/gdb/gdb.c:26 inf 1
>
> will remain the same all the time, but the objects (both in Python and
> their GDB backends) in gdb.breakpoints()[0].locations() will be
> deleted and re-created at various moments (cf. previous discussions).
> So far, I couldn't really understand what's the rational behind that
> ...
>
> [I'm not sure how well you know this part of GDB, please don't
> hesitate to correct me if I'm wrong]

Yikes, missed that.
GDB internals are free to change at will, the python API is the exact opposite.
This is the kind of API addition that gives me pause.
Once it's in we're stuck.

I once wrote a doc on managing GDB features/APIs, loosely based on
what I could remember of the SystemV ABI.
The ABI elements had three stages in their life, though I don't fully
remember the details.
IMO, IWBN to have something like it for our Python API.
In particular, one could remove something, after a preset amount of
time had passed since the announcement of its pending removal.
And one could add new things without promising they'll be there forever.
That scheme is old, and my memory is sketchy.
My point is that today we don't have anything other than: "Once it's
in we're stuck." [AFAICT]

For the case at hand, IWBN to have some experience with whether this
addition works well enough that we want to keep it.
And it's not clear we can determine that without some real world use of it.

  reply	other threads:[~2012-01-11 19:25 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-08 10:17 Kevin Pouget
2011-12-08 13:39 ` Phil Muldoon
2011-12-08 14:28   ` Kevin Pouget
2011-12-08 14:56 ` Phil Muldoon
2011-12-09 13:49   ` Kevin Pouget
2011-12-09 14:15     ` Phil Muldoon
2011-12-13 15:55       ` Kevin Pouget
2012-01-09 11:47         ` Kevin Pouget
2012-01-09 17:23           ` Eli Zaretskii
2012-01-10 15:09             ` Kevin Pouget
2012-01-10 16:03               ` Kevin Pouget
2012-01-10 17:25               ` Eli Zaretskii
2012-01-11 10:16                 ` Kevin Pouget
2012-01-11 10:27                   ` Eli Zaretskii
2012-01-10 22:24               ` Doug Evans
2012-01-11  9:05                 ` Kevin Pouget
2012-01-11 19:45                   ` Doug Evans [this message]
2012-01-27 13:04                     ` Kevin Pouget
2012-03-30 19:51                       ` Tom Tromey
2012-04-03 10:35                         ` Kevin Pouget
2012-04-03 12:15                           ` Phil Muldoon
2012-04-03 14:43                             ` Paul_Koning
2012-04-04  8:36                               ` Kevin Pouget
2012-05-09  7:18                                 ` Kevin Pouget
2012-04-05 16:27                           ` 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='CADPb22RTZmwQ3VsrYYw--o=qUp+3_FOT6Sx6BU5dkcP9uGKmkw@mail.gmail.com' \
    --to=dje@google.com \
    --cc=eliz@gnu.org \
    --cc=gdb-patches@sourceware.org \
    --cc=kevin.pouget@gmail.com \
    --cc=pmuldoon@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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).