This is the mail archive of the
gdb-patches@sourceware.cygnus.com
mailing list for the GDB project.
Re: [MAINT/PATCH] ChangeLog et.al.: Revert whitespace changes
- To: gdb-patches at sourceware dot cygnus dot com
- Subject: Re: [MAINT/PATCH] ChangeLog et.al.: Revert whitespace changes
- From: Andrew Cagney <ac131313 at cygnus dot com>
- Date: Mon, 03 Apr 2000 18:03:23 +1000
- Organization: Cygnus Solutions
- References: <38DEF05E.9D2E79AC@cygnus.com> <200003312332.SAA07205@indy.delorie.com> <38E7FE76.DCCD0E1A@cygnus.com>
Andrew Cagney wrote:
>
> Eli Zaretskii wrote:
> >
> > > I've just committed the attatched. It reverts a string of arbitrary
> > > whitespace changes made to various files including ChangeLogs.
> > >
> > > For ChangeLogs, they should be prepend-only.
> >
> > Sorry, I don't understand some of these changes, like this one:
> >
> > > *************** Fri Mar 17 11:06:59 2000 Philippe De Mu
> > > *** 429,435 ****
> > > * config/i386/tm-linux.h (I386_LINUX_SIGTRAMP): No need to define
> > > this any more, since we're not enabling OS-specific code in a
> > > OS-independent file.
> > > !
> > > 2000-03-16 Eli Zaretskii <eliz@is.elta.co.il>
> > >
> > > * Makefile.in (go32-nat.o): Add prerequisites.
> > > --- 433,439 ----
> > > * config/i386/tm-linux.h (I386_LINUX_SIGTRAMP): No need to define
> > > this any more, since we're not enabling OS-specific code in a
> > > OS-independent file.
> > > !
> > > 2000-03-16 Eli Zaretskii <eliz@is.elta.co.il>
> > >
> > > * Makefile.in (go32-nat.o): Add prerequisites.
> >
> > The only change here is that in the new version, there's extra
> > whitespace in an otherwise empty line. It's quite possible that some
> > editor (such as Emacs) removed the whitespace when somebody pressed
> > M-q to reformat the paragraph.
>
> It is more than just M-q - I use that all the time and yet I don't
> commit arbitrary white space changes. What I assume is happening is an
> editor (probably emacs) has has been configured to strip/modify white
> space changes in a arbitrary way when ever it touches a file.
More on this. The M-q (fill-paragraph) that I use (ok its old - 19.34.1
:-/) likes to prefix each line with:
<TAB>* Makefile.in (CC_FOR_TARGET): Add new winsup directory
<SPACE><TAB>structure stuff to -L library search.
<TAB>(CXX_FOR_TARGET): Ditto.
I suspect that another likes to indent it as:
<TAB>* Makefile.in (CC_FOR_TARGET): Add new winsup directory<SPACE>
<TAB>structure stuff to -L library search.<SPACE>
Since there is no requirement to use a specific change-log-mode (or even
emacs - ed is the editor of choice ..) we have to grin and bare this.
Andrew