[MPlayer-dev-eng] [PATCH] bswap.h which takes care of cpu's > 386 by using different asm commands

Felix Buenemann atmosfear at users.sourceforge.net
Mon Aug 5 14:50:19 CEST 2002


On Sunday 04 August 2002 20:58, Falk Hueffner wrote:
> > -#define bswap_64(x) \
> > -     (__extension__						\
> > -      ({ union { __extension__ unsigned long long int __ll;	\
> > -                 unsigned long int __l[2]; } __w, __r;		\
> > -         __w.__ll = (x);					\
> > -         __r.__l[0] = bswap_32 (__w.__l[1]);			\
> > -         __r.__l[1] = bswap_32 (__w.__l[0]);			\
> > -         __r.__ll; }))
> > -#endif	/* !ARCH_X86 */
>
> By the way, this is broken, too, since long int is 64 bit on some
> architectures (It was fixed in glibc some years ago). Please somebody
> s/unsigned long int/unsigned int/.
ok I verified and changed, please check if it's ok for your (cvs).

>         Falk

-- 
Best Regards,
        Atmos
____________________________________________
- MPlayer Developer - http://mplayerhq.hu/ -
____________________________________________



More information about the MPlayer-dev-eng mailing list