This is the mail archive of the
binutils@sources.redhat.com
mailing list for the binutils project.
Re: patch for binutils include/ansidecl.h.
- From: "David O'Brien" <obrien at FreeBSD dot org>
- To: DJ Delorie <dj at delorie dot com>
- Cc: eraxxon at alumni dot rice dot edu, binutils at sources dot redhat dot com
- Date: Sun, 22 Sep 2002 20:40:58 -0700
- Subject: Re: patch for binutils include/ansidecl.h.
- Organization: The NUXI BSD Group
- References: <3D8B6805.20306@alumni.rice.edu> <200209201835.g8KIZbX06187@envy.delorie.com>
- Reply-to: obrien at FreeBSD dot org
On Fri, Sep 20, 2002 at 02:35:37PM -0400, DJ Delorie wrote:
> > 2002-09-20 Nathan Tallent <eraxxon@alumni.rice.edu>
> > * ansidecl.h [__STDC__]: The Compaq C++ compiler, unlike many
> > other C++ compilers, does not define __STDC__, though it acts as
> > if this was so. Add (__alpha && __cplusplus) to the list of
> > platform compilers that may look, smell and act like __STDC__
> > but that may not define it.
>
> The ChangeLog entry should say what changed, not why.
That is a real limitation and annoying rule. It is often impossible to
find out *WHY* a change was done. Which is a Bad Thing. I can read the
"cvs diff -r X -r X+1" to see the what. I really wish GNU ChangeLog
entries would grow the *WHY*, or the culture be changed to tell the *WHY*
rather than just repleat the ChangeLog entry in the CVS commit log.