[PATCH] Fix includes of sparseset.*

Peter Bergner bergner@vnet.ibm.com
Tue Dec 4 15:01:00 GMT 2007


On Sat, 2007-12-01 at 13:41 +0100, Bernhard Fischer wrote:
> On Wed, Nov 28, 2007 at 10:44:12AM -0700, Tom Tromey wrote:
> >>>>>> "Bernhard" == Bernhard Fischer <rep.dot.nop@gmail.com> writes:
> >
> >Bernhard> 	* sparseset.h: Include config.h before system.h
> >Bernhard> 	* sparseset.c: Remove inclusion of libiberty.h
> >
> >I think it is a bit odd to include config.h from a header.
> >
> >Instead, I think as a rule every .c file should include config.h as
> >the first header.
> 
> Updated patch attached. Bootstrapped and regtested on i386-linux-gnu
> without any new regression. Ok for trunk?

This bootstrapped for me on powerpc64-linux with no errors.  I ran the
testsuite, but I don't have baseline results to compare with it at the
moment.  I'm not sure I can approve this even though I wrote the files,
but it looks good to me.

Peter





More information about the Gcc-patches mailing list