On Wed, Apr 8, 2009 at 13:21, Jack Stone <jwjstone@xxxxxxxxxxx> wrote: > Remove uneeded void casts > > Signed-Off-By: Jack Stone <jwjstone@xxxxxxxxxxx> > --- > lib/inflate.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/lib/inflate.c b/lib/inflate.c > index 1a8e8a9..4b672f9 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; > > if (free_mem_end_ptr && malloc_ptr >= free_mem_end_ptr) > @@ -481,7 +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))) == > + if ((q = malloc((z + 1)*sizeof(struct huft))) == > (struct huft *)NULL) Thats an unneeded cast too. (After that, the NULL can move up one line.) Bert > { > if (h) -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html