This is the mail archive of the
gdb-patches@sourceware.cygnus.com
mailing list for the GDB project.
Re: [PATCH RFC] PARAMS elimination - phase 2
- To: Eli Zaretskii <eliz at is dot elta dot co dot il>
- Subject: Re: [PATCH RFC] PARAMS elimination - phase 2
- From: Michael Snyder <msnyder at cygnus dot com>
- Date: Wed, 31 May 2000 08:00:52 -0700
- CC: kevinb at cygnus dot com, gdb-patches at sourceware dot cygnus dot com
- Organization: Cygnus Solutions
- References: <1000530232352.ZM5368@ocotillo.lan> <200005310705.DAA08769@indy.delorie.com>
- Reply-To: msnyder at cygnus dot com
Eli Zaretskii wrote:
> [snip]
> > -void (*target_overlay_update) PARAMS ((struct obj_section *))
> > -= simple_overlay_update;
> > +void (*target_overlay_update) (struct obj_section *) = simple_overlay_update;
> [snip]
> > -static struct partial_symbol *lookup_partial_symbol PARAMS
> > - ((struct partial_symtab *, const char *,
> > - int, namespace_enum));
> > +static struct partial_symbol *lookup_partial_symbol
> > + (struct partial_symtab *, const char *, int, namespace_enum);
> [snip]
> > - struct target_ops
> > - *DONT_USE; /* formerly to_next */
> > + struct target_ops *DONT_USE; /* formerly to_next */
>
> The above are a few of the examples of such a gratuitous
> reformatting. I think they are evil and should be avoided at all
> costs. They make it a PITA to apply patches sent by people who have
> sources before the reformatting. Introducing changes that merely
> reshuffle white space really doesn't make sense to me.
>
> I vote against those whitespace changes (even though these are not
> files I'm responsible for).
I would tend to disagree. In the first place, I like the
single-line declarations better when there is no reason to
break them up into two lines. In the second place, we agreed
that the output of indent, suitably tweaked if necessary to
conform to the GNU coding standard, would be our rule.
Michael