[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [SCM] GNU M4 source repository branch, branch-1.4, updated. v1.4.16-
From: |
Gary V. Vaughan |
Subject: |
Re: [SCM] GNU M4 source repository branch, branch-1.4, updated. v1.4.16-3-gfb132bf |
Date: |
Sat, 30 Apr 2011 14:16:47 +0700 |
User-agent: |
Mutt/1.5.20 (2009-06-14) |
Hi Eric,
On Fri, Apr 29, 2011 at 09:56:08PM +0000, Eric Blake wrote:
> - Log -----------------------------------------------------------------
> commit fb132bf05f993b98f8bce6dc681dd34cacc40a39
> Author: Eric Blake <address@hidden>
> Date: Fri Apr 29 15:50:31 2011 -0600
>
> m4_defn: avoid compiler warning
>
> argc is an int, therefore by definition, it is <= INT_MAX.
>
> * src/builtin.c (m4_defn): Nuke redundant condition.
>
> Signed-off-by: Eric Blake <address@hidden>
>
> -----------------------------------------------------------------------
>
> diff --git a/src/builtin.c b/src/builtin.c
> index 632ef79..80f2024 100644
> --- a/src/builtin.c
> +++ b/src/builtin.c
> @@ -894,7 +894,7 @@ m4_defn (struct obstack *obs, int argc, token_data **argv)
> if (bad_argc (argv[0], argc, 2, -1))
> return;
>
> - assert (0 < argc && argc <= INT_MAX);
> + assert (0 < argc);
> for (i = 1; i < (unsigned) argc; i++)
> {
> const char *arg = ARG((int) i);
Since argc is immediately cast to an unsigned int, I think the test was
supposed to be:
assert (0 < argc && argc <= UINT_MAX);
Cheers,
--
Gary V. Vaughan (gary AT gnu DOT org)
pgpHaSi86Scnv.pgp
Description: PGP signature