[SDL] [patch] add support for arm/thumb and superh to endian

Mike Frysinger vapier at gentoo.org
Mon Sep 5 22:14:10 PDT 2005


On Thursday 01 September 2005 09:02 pm, Simon Roby wrote:
> On 9/1/05, Mike Frysinger <vapier at gentoo.org> wrote:
> > find attached a patch to add support for detecting endian on superh and
> > arm/thumb ... also, ive incorporated a patch from Fedora which will
> > gather the correct endian on a linux host from the system endian.h
> > instead of just trying to maintain an ever-growing list of architectures
>
> Why not just use autoconf's AC_C_BIGENDIAN macro? Should be more portable.

the comments in the file say this is done via preprocessor macros instead of 
configure checks on purpose

if you have a configure check, that means the endian header file (which is 
currently a 'public' header file for any old application to include) would be 
dependent on similar preprocessor's existing in every build system ... you 
could make the configure script generate the endian header file so that the 
BIGENDAIN define is included in it directly, but then you tie the header 
files to the host system which libsdl was built for, and that too has its 
disadvantages
-mike




More information about the SDL mailing list