[pkg-fgfs-crew] Bug#572428: Bug#572428: Bug#572428: BTS Link

[Nobuhiro Iwamatsu]

Hi,

2011/1/13 Ove Kåven <ovek at arcticnet.no>:
> Den 12. jan. 2011 11:40, skrev Nobuhiro Iwamatsu:
>> OK,
>> I made the patch which supported SuperH only.
>> I attach it. Could you check this patch?
>
> According to my web searches, there may be some SuperH toolchains out
> there (like GNUPro, it seems) which define __LITTLE_ENDIAN__, but not
> __BIG_ENDIAN__. Thus, to be safe, I suppose it might be better with
> something like

__BIG_ENDIAN__ is defined in builtin_define.
When you have only a compiler of little, I may see only __LITTLE_ENDIAN__.

However, I think that your patch is safer as you point it out.
Thank you.

>
> --- a/simgear/nasal/naref.h
> +++ b/simgear/nasal/naref.h
> @@ -12,11 +12,13 @@
>  # define NASAL_NAN64
>  #elif defined(_M_IX86) || defined(i386) || defined(__x86_64) || \
>       defined(__ia64__) || defined(_M_IA64) || defined(__ARMEL__) || \
> -      defined(_M_X64) || defined(__alpha__)
> +      defined(_M_X64) || defined(__alpha__) || \
> +         (defined(__sh__) && defined(__LITTLE_ENDIAN__))
>  # define NASAL_LE
>  #elif defined(__sparc) || defined(__ppc__) || defined(__PPC) || \
>       defined(__mips) || defined(__ARMEB__) || \
> -      defined(__hppa__) || defined(__s390__) || defined(__s390x__)
> +      defined(__hppa__) || defined(__s390__) || defined(__s390x__) || \
> +         (defined(__sh__) && !defined(__LITTLE_ENDIAN__))
>  # define NASAL_BE
>  #else
>  # error Unrecognized CPU architecture



-- 
Nobuhiro Iwamatsu
   iwamatsu at {nigauri.org / debian.org}
   GPG ID: 40AD1FA6