Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Joel Brobecker <brobecker@gnat.com>
To: Andrew Cagney <cagney@gnu.org>
Cc: Eli Zaretskii <eliz@gnu.org>,
	kettenis@gnu.org, gdb-patches@sources.redhat.com
Subject: Re: [patch] Deprecate XM_FILE and TM_FILE
Date: Mon, 13 Sep 2004 21:30:00 -0000	[thread overview]
Message-ID: <20040913213015.GC5843@gnat.com> (raw)
In-Reply-To: <41460B78.90005@gnu.org>

> I've split this patch in two and committed just the TM_FILE stuff.  As 
> for the XM_FILE changes (and those 3 definitions), consider that tabled.

It seems to me that the whole discussion between Eli and yourself has
been beaten to death. We're basically stuck in a disagrement where both
point of views have their merit.

I think it's time all global maintainers get involved in this discussion
and make a decision. Once the decision is taken, it needs to be
documented (gdb.texinfo for instance) so that people can refer to it.

As a developper, I personally dislike to have to check the ARI everytime
I use anything in GDB for fear of using something deprecated. So marking
each instance as explicitly deprecated directly in the code is a good
move. Two questions were asked and need to be answered. I am adding my
proposed answers, as a starting point for your discussion:

  1. When can some code be declared deprecated?

     IMO, some code should be declared deprecated when it has been
     recognized that it should no longer be used in new changes.
     It means that some code can be identified as deprecated before
     a replacement has been implemented.

     There is a judgement call to make, obviously, as we don't want to
     deprecate a central piece of GDB that makes it impossible for
     somebody to submit a new port for instance without doing man-years
     of work required to implement an alternate to the deprecated
     feature.

  2. How to identify deprecated code?

     Deprecated code should be explicitly marked as such directly
     in the code, to avoid any accidental future usage, by prepending
     "depreated_" to the entity names.
     
     Deprecated code can only be removed when no longer used. There can
     be no time limit imposed between the time some code is deprecated,
     and the time when it is removed.

     (the alternate solution suggested by Eli is the ARI)

You should decide how the discussion will be held (privately or on
gdb-patches?), and whether it should include the steering committee
or not.

Happy discussion.

-- 
Joel


  reply	other threads:[~2004-09-13 21:30 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-09-02 18:30 Andrew Cagney
2004-09-02 20:26 ` Eli Zaretskii
2004-09-03 16:18   ` Andrew Cagney
2004-09-04 12:00     ` Eli Zaretskii
2004-09-04 14:27       ` Andrew Cagney
2004-09-04 16:28         ` Eli Zaretskii
2004-09-04 23:20           ` Andrew Cagney
2004-09-05  4:17             ` Eli Zaretskii
2004-09-09 16:05               ` Andrew Cagney
2004-09-09 19:31                 ` Eli Zaretskii
2004-09-09 20:26                   ` Andrew Cagney
2004-09-09 21:15                     ` Eli Zaretskii
2004-09-09 21:26                       ` Joel Brobecker
2004-09-10  9:35                         ` Eli Zaretskii
2004-09-10 12:41                           ` Mark Kettenis
2004-09-10 16:32                             ` Eli Zaretskii
2004-09-12 18:07                               ` Andrew Cagney
2004-09-12 18:36                                 ` Eli Zaretskii
2004-09-13 15:35                                   ` Andrew Cagney
2004-09-13 19:48                                     ` Eli Zaretskii
2004-09-13 21:07                                       ` Andrew Cagney
2004-09-13 21:30                                         ` Joel Brobecker [this message]
2004-09-24 22:06                                           ` Andrew Cagney
2004-09-15 12:15                                         ` Eli Zaretskii
2004-09-15 15:54                                           ` 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=20040913213015.GC5843@gnat.com \
    --to=brobecker@gnat.com \
    --cc=cagney@gnu.org \
    --cc=eliz@gnu.org \
    --cc=gdb-patches@sources.redhat.com \
    --cc=kettenis@gnu.org \
    /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