lkml.org 
[lkml]   [2009]   [Apr]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: [PATCH 20/56] inflate: Remove void casts
    From
    On Wed, Apr 8, 2009 at 2:54 PM, Jack Stone <jwjstone@fastmail.fm> wrote:
    > Heres the fixed patch
    >
    > Thanks,
    >
    > Jack
    >
    > --
    >
    > inflate: Remove void casts
    >
    > From: Jack Stone <jwjstone@fastmail.fm>
    >
    > Remove uneeded void casts
    >
    > Signed-Off-By: Jack Stone <jwjstone@fastmail.fm>
    > ---
    >
    >  lib/inflate.c |    5 ++---
    >  1 files changed, 2 insertions(+), 3 deletions(-)
    >
    > diff --git a/lib/inflate.c b/lib/inflate.c
    > index 1a8e8a9..1eb0cd8 100644
    > --- a/lib/inflate.c
    > +++ b/lib/inflate.c
    > @@ -249,7 +249,7 @@ static void *malloc(int size)
    >
    >        malloc_ptr = (malloc_ptr + 3) & ~3;     /* Align */
    >
    > -       p = (void *)malloc_ptr;
    > +       p = malloc_ptr;
    >        malloc_ptr += size;

    Won't this cause a "assignment makes pointer from integer without a
    cast" warning?

    >        if (free_mem_end_ptr && malloc_ptr >= free_mem_end_ptr)
    > @@ -481,8 +481,7 @@ DEBG1("3 ");
    >         z = 1 << j;             /* table entries for j-bit table */
    >
    >         /* allocate and link in new table */
    > -        if ((q = (struct huft *)malloc((z + 1)*sizeof(struct huft))) ==
    > -            (struct huft *)NULL)
    > +        if ((q = malloc((z + 1)*sizeof(struct huft))) == NULL)
    >         {
    >           if (h)
    >             huft_free(u[0]);
    > --
    > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    > the body of a message to majordomo@vger.kernel.org
    > More majordomo info at  http://vger.kernel.org/majordomo-info.html
    > Please read the FAQ at  http://www.tux.org/lkml/
    >
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2009-04-08 16:09    [W:3.695 / U:0.332 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site