[Ffmpeg-devel] PATCH Blackfin optimized byte swapping mechanism
Tue Apr 24 03:46:09 CEST 2007
Michael Niedermayer writes:
> not everyone duplicated these 2 bswap functions, ARCH_ARM does not for
> also, just because the code in svn is a mess does not mean its ok to add
> more mess
> and, iam rejecting patches which are messy mostly independant upon how
> difficult it is to implement a clean solution, so if it would require
> rewriting bswap.h then so be it but its not needed here, simply doing
> what ARCH_ARM does should work even if its not beautifull ...
> if you want to cleanup bswap.h that is certainly welcome too, maybe
> a bswap.h which contains just the c functions under #ifndef FOOBAR16/32/64
> and then a bswap_arm.h, ...
I see your point and it makes good sense and is a much simpler patch
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 608 bytes
Desc: not available
More information about the ffmpeg-devel