Re: [commit] Deprecate remaining STREQ uses

Andrew Cagney
2003-12-01T21:25:55Z

[-- Attachment #1: Type: text/plain, Size: 979 bytes --]

> On Mon, Dec 01, 2003 at 09:07:32PM +0200, Eli Zaretskii wrote:
>
>> > Date: Mon, 01 Dec 2003 10:47:01 -0500
>> > From: Andrew Cagney <cagney@gnu.org>
>
>> > >
>> > > But that's precisely why we have the patch review and approval
>> > > procedure, right? Maintainers who approve patches are supposed to
>> > > prevent code that uses deprecated machinery from being added.
>
>> >
>> > Very true. Explicit deprecation is a tool for making that part of the
>> > maintainer and contributor task far easier. Instead of wasting time
>> > trying to track and find all the things being eliminated, the
>> > contributor and reviewer can simply keep an eye out for deprecated in
>> > their patches
>
>>
>> I'm not convinced that detecting STREQ is harder than detecting
>> DEPRECATED_STREQ.
>
>
> Neither am I... Andrew, how would you feel about a central (in the
> source tree) list of deprecated objects instead?

I see you didn't reply to the attached e-mail.

Andrew

[-- Attachment #2: mailbox-message://ac131313@movemail/fsf/gdb/patches#8370148 --]
[-- Type: message/rfc822, Size: 4019 bytes --]

From: Andrew Cagney <cagney@gnu.org>
To: Daniel Jacobowitz <drow@mvista.com>
Cc: gdb-patches@sources.redhat.com
Subject: Re: [commit] Deprecate remaining STREQ uses
Date: Mon, 24 Nov 2003 17:08:54 -0500
Message-ID: <3FC28176.5020908@gnu.org>

> At least one now :) There are a number of other solutions to this.
> Have you considered making the ARI mail contributors for certain
> (low-false-positive) categories? Like, for instance, this one. The
> gcc-regression mailing list has several scripts to pull the ChangeLog
> entries since the last run and mail victims. It's extremely effective.

I find the GCC script anything but effective. I get spammed everytime I
commit something to GCC - a very negative experience for an infrequent
GCC committer. I've now been conditioned into ignoring that mail :-(

Contrast that to -Werror (yes ok, it isn't a requirement) and
gdb_mbuild.sh. By encouraging their use we make it possible for people
to address the problems _before_ they become an issue. That way the
contributor and maintainer don't even need to discuss them. For
something like the ARI to be mainlined, it would need to be integrated
into the build process in a way that didn't leave the user confused (a
standard build would have to be 100% warning free - something that at
present is impossible to achieve).

enjoy,
Andrew