flac dev - Jul 2014 - [PATCH] two fixes

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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 1_mklite.patch
Type: application/octet-stream
Size: 426 bytes
Desc: not available
Url :
http://lists.xiph.org/pipermail/flac-dev/attachments/20140709/c6622c31/attachment.obj
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 2_endswap.patch
Type: application/octet-stream
Size: 511 bytes
Desc: not available
Url :
http://lists.xiph.org/pipermail/flac-dev/attachments/20140709/c6622c31/attachment-0001.obj

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/