On Wed, Apr 01, 2015 at 06:14:06PM +0200, Marc Kleine-Budde wrote: > Signed-off-by: Marc Kleine-Budde <[email protected]> > --- > scripts/imx/imx-image.c | 15 ++++++++------- > 1 file changed, 8 insertions(+), 7 deletions(-) > > diff --git a/scripts/imx/imx-image.c b/scripts/imx/imx-image.c > index 1f37fe20bcc0..2fa84b1a1d53 100644 > --- a/scripts/imx/imx-image.c > +++ b/scripts/imx/imx-image.c > @@ -15,17 +15,18 @@ > * GNU General Public License for more details. > * > */ > -#include <stdio.h> > -#include <unistd.h> > +#include <endian.h> > +#include <errno.h> > +#include <fcntl.h> > #include <getopt.h> > -#include <stdlib.h> > #include <stdint.h> > +#include <stdio.h> > +#include <stdlib.h> > #include <string.h> > -#include <errno.h> > -#include <sys/types.h> > +#include <unistd.h> > + > #include <sys/stat.h> > -#include <fcntl.h> > -#include <endian.h> > +#include <sys/types.h>
I sometimes sort the includes by length which also looks very nice ;) We don't have a policy how includes should be sorted, so I prefer to keep them like they are. Also sometimes the include order matters since not all include files include everything they need themselves, This of course should be fixed, but forcing you to fix it before you can add new files with sorted includes causes additional pain which I think is unnecessary Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list [email protected] http://lists.infradead.org/mailman/listinfo/barebox
