[flac-dev] [PATCH] two fixes

Erik de Castro Lopo mle+la at mega-nerd.com
Sun Jul 13 03:37:51 PDT 2014


lvqcl wrote:

> make -f Makefile.lite cannot compile test_libFLAC/endswap.c
> because CPU_IS_LITTLE_ENDIAN definition is missing, and also
> complains about parentheses in ENDSWAP_16.
> 
> 
> 1) The first patch adds the definition of CPU_IS_LITTLE_ENDIAN into build/config.mk
> which allows to build test_libFLAC with Makefile.lite
> 
> 
> 2) Current ENDSWAP_16 macro
>      (((((x) >> 8) & 0xFF) + ((x) & 0xFF) << 8))
> adds (x >> 8) & 0xFF with x & 0xFF and then l-shifts the result by 8.
> It doesn't looks right. The second patch fixes this.

Applied. Thanks.

Erik
-- 
----------------------------------------------------------------------
Erik de Castro Lopo
http://www.mega-nerd.com/


More information about the flac-dev mailing list