[U-Boot] [PATCH v2 13/25] lib/zlib: sparse fixes

Wolfgang Denk wd at denx.de
Thu Nov 8 12:20:51 CET 2012


Dear Kim Phillips,

In message <1351553687-12469-14-git-send-email-kim.phillips at freescale.com> you wrote:
> define Z_NULL to (void *)0 include/u-boot/zlib.h to get rid of most of
> the NULL pointer warnings.
...
> -#define Z_NULL  0  /* for initializing zalloc, zfree, opaque */
> +#define Z_NULL  (void *)0  /* for initializing zalloc, zfree, opaque */

Don't we have to put parens around such an expression?


> --- a/lib/zlib/adler32.c
> +++ b/lib/zlib/adler32.c
> @@ -54,10 +54,7 @@
>  #endif
>  
>  /* ========================================================================= */
> -uLong ZEXPORT adler32(adler, buf, len)
> -    uLong adler;
> -    const Bytef *buf;
> -    uInt len;
> +uLong ZEXPORT adler32(uLong adler, const Bytef *buf, uInt len)

Hm... this code has been imported from another project.  This
modification makes updates (or even comparing what has changed in more
recent versions) much mor difficult, or even impossible
:-(

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
I have a very small mind and must live with it.    -- Edsger Dijkstra


More information about the U-Boot mailing list