Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Andrew Cagney <ac131313@redhat.com>
To: Daniel Jacobowitz <drow@mvista.com>
Cc: Mark Kettenis <kettenis@chello.nl>, gdb-patches@sources.redhat.com
Subject: Re: [rfa/i386] Make codestream deprecated?
Date: Wed, 08 Jan 2003 22:43:00 -0000	[thread overview]
Message-ID: <3E1CA974.6070807@redhat.com> (raw)
In-Reply-To: <20030108211935.GA22170@nevyn.them.org>

> On Wed, Jan 08, 2003 at 04:13:50PM -0500, Daniel Jacobowitz wrote:
> 
>> On Wed, Jan 08, 2003 at 04:03:33PM -0500, Andrew Cagney wrote:
> 
>> > 
> 
>> > >Besides: this is why you should _remove_ them, rather than just
>> > >commenting them, if you want them to go away.
> 
>> > 
>> > The implication here being that I'm not removing deprecated code?
> 
>> 
>> A heck of a lot slower than you're deprecating!  There are now over
>> 1200 references to deprecated constructs in GDB.  That's just a rough
>> estimate, mind.
>> 
>> I understand why release branches are bad points to compare against for
>> this, but since I have several handy:
>>   ~150 deprecated references in gdb 5.3
>>   ~30 in gdb 5.2
>> 
> 
>> > The simple reality is that it is not possible for a single individual 
>> > remove all the old code in a single hit.  It takes a group of people 
>> > co-operating, and it takes time.
> 
> 
> I feel the need to add:
> 
> If you are deprecating things single-handedly, what group of people do
> you expect to co-operate?  It's a matter of cleaning up after yourself.

No one is trying to do anything single handedly.  I'm just the luck one 
that gets to follow through some of the dirty work (and put up with the 
resultant flack :-).

Since 5.3 branched, several major changes have gone down, and one is 
still in progress:

- the complaint mechanism was replaced.
Kevin did the bulk of the work here and in the process fixed a 
significant number of bugs!

- the register cache was rewritten
Probably the single biggest nasty bit of work - registers[] - is no 
longer relevant.  The core code no longer uses it.  People are slowly 
expunging -tdep and -nat code of the remainging references.

- the frame's 15 year service
Cleanly integrating things like cfi-frame is made possible; sillyness 
such as re-analyzing a prolog N times can be stopped; the posability of 
having more than one active frame chain (one per thread, hint, hint) is 
real.

The immedate focus is obviously on finishing the new code so that that 
doesn't end up going the same way as many other half finished GDB 
projects (gdb, you'll have to agree, already has too many of these).  To 
that very end, I've finished making `struct frame_info' opaque and I can 
start changing its internals.

However, like others (including you) I also spend time expunging 
deprecated methods.

enjoy,
Andrew



  reply	other threads:[~2003-01-08 22:43 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-12-01 16:39 Andrew Cagney
2002-12-01 17:02 ` Daniel Jacobowitz
2002-12-01 17:51   ` Andrew Cagney
2002-12-07 10:16 ` Mark Kettenis
2003-01-08 20:18   ` Andrew Cagney
2003-01-08 20:31     ` Daniel Jacobowitz
2003-01-08 21:03       ` Andrew Cagney
2003-01-08 21:13         ` Daniel Jacobowitz
2003-01-08 21:19           ` Daniel Jacobowitz
2003-01-08 22:43             ` Andrew Cagney [this message]
2003-01-30  3:31 ` Andrew Cagney

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=3E1CA974.6070807@redhat.com \
    --to=ac131313@redhat.com \
    --cc=drow@mvista.com \
    --cc=gdb-patches@sources.redhat.com \
    --cc=kettenis@chello.nl \
    /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