Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: Andrew Cagney <ac131313@ges.redhat.com>
To: David Carlton <carlton@math.stanford.edu>
Cc: gdb <gdb@sources.redhat.com>, Joel Brobecker <brobecker@ACT-Europe.FR>
Subject: Re: list of GNU indent versions in gdb_indent.sh
Date: Thu, 12 Sep 2002 13:01:00 -0000	[thread overview]
Message-ID: <3D80F28B.6010409@ges.redhat.com> (raw)
In-Reply-To: <ro1k7lr2eeg.fsf@jackfruit.Stanford.EDU>

> I noticed that gdb_indent.sh complained at me because I'm using GNU
> indent 2.2.7 instead of 2.2.6.  I ignored the warning, but of course
> that's something I don't like to do.  Does anybody have any reason to
> believe that GNU indent 2.2.7 or 2.2.8 isn't a reasonable version as
> well?  If not, I'll look through the ChangeLogs/NEWS/whatever the
> appropriate file is for GNU indent for those versions and, assuming
> they don't do anything strange, modify gdb_indent.sh to treat them as
> "reasonable".

The check is to ensure that everyone uses the same version of indent. 
That way ``obvious'' re-indentation doesn't cause unnecessary entropy.
(BTW, re-indentation isn't normally a reason for adding a (C) 2002 to a 
file.  For symtab.h, it had seen other [minor] changes so it doesn't hurt.)

Can you please find out exactly what was changed?  By definition (baring 
a really bad bug) GDB's indentation style is determined by the output 
(no matter how bad :-) of [current] GNU indent with out any parameters 
(ignoring -T).  When a new indent becomes available we should move to it.

I'll tweak the script so that it simply rejects wrong indent versions.

--

Joel,

W.r.t. indent --<white-space-instead-of-tab>, that is somebody elses 
problem :-)  GDB simply lives with the output of  GNU indent -T ..., no 
matter how bad it is :-)  If EMACS/indent ever change their default, 
we'll change as well.

enjoy,
Andrew



  reply	other threads:[~2002-09-12 20:01 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-09-12 12:31 David Carlton
2002-09-12 13:01 ` Andrew Cagney [this message]
     [not found]   ` <ro1ptvi29sq.fsf@jackfruit.Stanford.EDU>
2002-09-12 14:21     ` David Carlton
2002-09-12 14:24       ` David Carlton
2002-09-19 16:10     ` 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=3D80F28B.6010409@ges.redhat.com \
    --to=ac131313@ges.redhat.com \
    --cc=brobecker@ACT-Europe.FR \
    --cc=carlton@math.stanford.edu \
    --cc=gdb@sources.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