[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    Re: [PATCH] lzop.c: remove unneeded variable
From:       Denys Vlasenko <vda.linux () googlemail ! com>
Date:       2015-10-28 14:34:31
Message-ID: CAK1hOcNRuVeq3TTFxr9cXbsCrgte1KyhMMEBaGP1OmTUWGyYbQ () mail ! gmail ! com
[Download RAW message or body]

Applied, thanks!

On Wed, Oct 28, 2015 at 2:32 PM, Maxin B. John <maxin.john@intel.com> wrote:
> remove unneeded variable 'ok'
>
> Signed-off-by: Maxin B. John <maxin.john@intel.com>
> ---
>  archival/lzop.c | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/archival/lzop.c b/archival/lzop.c
> index 73d11a7..5bc1b34 100644
> --- a/archival/lzop.c
> +++ b/archival/lzop.c
> @@ -650,7 +650,6 @@ static NOINLINE smallint lzo_compress(const header_t *h)
>         uint32_t d_adler32 = ADLER32_INIT_VALUE;
>         uint32_t d_crc32 = CRC32_INIT_VALUE;
>         int l;
> -       smallint ok = 1;
>         uint8_t *wrk_mem = NULL;
>
>         if (h->method == M_LZO1X_1)
> @@ -732,7 +731,7 @@ static NOINLINE smallint lzo_compress(const header_t *h)
>         free(wrk_mem);
>         free(b1);
>         free(b2);
> -       return ok;
> +       return 1;
>  }
>
>  static FAST_FUNC void lzo_check(
> @@ -761,7 +760,6 @@ static NOINLINE smallint lzo_decompress(const header_t *h)
>         uint32_t c_adler32 = ADLER32_INIT_VALUE;
>         uint32_t d_adler32 = ADLER32_INIT_VALUE;
>         uint32_t c_crc32 = CRC32_INIT_VALUE, d_crc32 = CRC32_INIT_VALUE;
> -       smallint ok = 1;
>         uint8_t *b1;
>         uint32_t mcs_block_size = MAX_COMPRESSED_SIZE(block_size);
>         uint8_t *b2 = NULL;
> @@ -865,7 +863,7 @@ static NOINLINE smallint lzo_decompress(const header_t *h)
>         }
>
>         free(b2);
> -       return ok;
> +       return 1;
>  }
>
>  /**********************************************************************/
> --
> 2.4.0
>
> _______________________________________________
> busybox mailing list
> busybox@busybox.net
> http://lists.busybox.net/mailman/listinfo/busybox
_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox
[prev in list] [next in list] [prev in thread] [next in thread]