[flac-dev] [PATCH] two fixes

lvqcl lvqcl.mail at gmail.com
Wed Jul 9 03:54:31 PDT 2014


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 


More information about the flac-dev mailing list