This is the mail archive of the gdb-patches@sourceware.cygnus.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: [PATCH RFA] Remove PARAMS from gdb source files


Kevin Buettner wrote:

> After this patch is applied, there are still around 260 occurrences
> of PARAMS in the gdb source files.  This is a reasonable number to
> do by hand, possibly with the assistance of a script similar to the
> one I used for generating the patch below.  I propose that we first
> make the changes contained in the patch below and then clean the rest
> up in later passes.
> 
> I'm not sure how to go about getting approval for a change this
> massive.  I guess I'll wait about a week for objections, complaints,
> etc. and then leave it up to Andrew for a final thumbs up or thumbs
> down.

I'd just set a firm date/time-range and then, in it goes.  Provided
everyone has a few days warning and knows where the cut over date is
there shouldn't be problems.

What time? Well the best is likely to be around midnight Sunday GMT
(that happens to be a normal Monday morning work day for me :-)  I'm
willing to run the script for you or you can set a time more convenient
for your self and run it then (in truth there is never a good time :-).

Having indent (no matter how bad its output :-) is also important.  If
someone gets stuck with a header file that got ripped apart they can
re-sync it using indent.

Thanks for this!

	Andrew

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]