Re: [patch] header file re-ordering.

[Jeff Law <law@redhat.com>]

On 10/08/2015 07:37 AM, Andrew MacLeod wrote:
> On 10/07/2015 06:02 PM, Jeff Law wrote:
>
> However, the tool has been run, and I've made the minor adjustments
> required to the source files to make it work.  (ie, a few multi-line
> comments and the fact that mul-tables.c is generated on the tile* targets.
>
> So this is what it should look like.  I used -cp.    Other languages are
> bootstrapping, and I have yet to build all the targets... that'll just
> take a day.   Be nice if ada worked tho.
>
> I can run the reduction tool over the weekend (its a long weekend here
> :-) on this if you want...  the other patch is a couple of weeks out of
> date anyway now.
So I'm playing with this stuff a little.  I was surprised to see that 
the reordering script also removes duplicates.

For some dumb reason I thought that functionality was part of the header 
file reducer, but that's only concerned with removing stuff that's 
unnecessary.

Anyway, just surprised me.  Not sure if it's worth splitting that 
functionality out or making it conditional on a flag is worth it.

It certainly helps in that I won't look at the changes and expect that 
headers are just reordered :-)

jeff