[U-Boot] [PATCH] Refresh LZMA-lib to v4.65

[Luigi Mantellini]

Hi Mike,

On Mon, Jul 27, 2009 at 7:40 PM, Mike Frysinger<vapier at gentoo.org> wrote:
> On Tuesday 21 July 2009 04:45:49 Luigi 'Comio' Mantellini wrote:
>> --- /dev/null
>> +++ b/lib_generic/lzma/LzmaDec.c
>> @@ -0,0 +1,1007 @@
>> +/* LzmaDec.c -- LZMA Decoder
>> +2008-11-06 : Igor Pavlov : Public domain */
>> +
>> +#include "LzmaDec.h"
>> +
>> +#include <string.h>
>
> there is no string.h in the source tree.  because of this, the .depend step
> fails:
> make[1]: Entering directory `/usr/local/src/u-boot/blackfin/lib_generic/lzma'
> LzmaDec.c:9:20: error: string.h: No such file or directory
>
> i know there's a hack in the Makefile:
> CFLAGS += -D_LZMA_PROB32 -I$(TOPDIR)/include/linux
>
> looks like this should be CPPFLAGS, not CFLAGS.  or fix the source and drop
> the -I hack.

I preferred to not touch the lzma sourcers. Anyway, on my system the
hack works fine... can you try to do a "make mrproper && make
your_board_config" before to compile?

>
>> --- a/lib_generic/lzma/LzmaTools.c
>> +++ b/lib_generic/lzma/LzmaTools.c
>> +    SizeT inProcessed;
>
> this variable isnt used in this function:
> LzmaTools.c: In function ‘lzmaBuffToBuffDecompress’:
> LzmaTools.c:64: warning: unused variable ‘inProcessed’

You are right :)

Thanks for your feedback!

ciao

luigi

> -mike
>





-- 
Luigi 'Comio' Mantellini
R&D - Software
Industrie Dial Face S.p.A.
Via Canzo, 4
20068 Peschiera Borromeo (MI), Italy

Tel.: +39 02 5167 2813
Fax: +39 02 5167 2459
web: www.idf-hit.com
mail: luigi.mantellini at idf-hit.com