lkml.org 
[lkml]   [2011]   [Mar]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] kstrtox: fix compile warnings in test
From
On Thu, Mar 24, 2011 at 19:51, Alexey Dobriyan <adobriyan@gmail.com> wrote:
> Fix the following warnings:
>
>  CC [M]  lib/test-kstrtox.o
> lib/test-kstrtox.c: In function 'test_kstrtou64_ok':
> lib/test-kstrtox.c:318: warning: this decimal constant is unsigned only in ISO C90
>        ...
>
> Noticed by Geert Uytterhoeven <geert@linux-m68k.org>
>
> Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>

Acked-by: Geert Uytterhoeven <geert@linux-m68k.org>

> ---
>
>  lib/test-kstrtox.c |   32 ++++++++++++++++----------------
>  1 file changed, 16 insertions(+), 16 deletions(-)
>
> --- a/lib/test-kstrtox.c
> +++ b/lib/test-kstrtox.c
> @@ -315,12 +315,12 @@ static void __init test_kstrtou64_ok(void)
>                {"65537",       10,     65537},
>                {"2147483646",  10,     2147483646},
>                {"2147483647",  10,     2147483647},
> -               {"2147483648",  10,     2147483648},
> -               {"2147483649",  10,     2147483649},
> -               {"4294967294",  10,     4294967294},
> -               {"4294967295",  10,     4294967295},
> -               {"4294967296",  10,     4294967296},
> -               {"4294967297",  10,     4294967297},
> +               {"2147483648",  10,     2147483648ULL},
> +               {"2147483649",  10,     2147483649ULL},
> +               {"4294967294",  10,     4294967294ULL},
> +               {"4294967295",  10,     4294967295ULL},
> +               {"4294967296",  10,     4294967296ULL},
> +               {"4294967297",  10,     4294967297ULL},
>                {"9223372036854775806", 10,     9223372036854775806ULL},
>                {"9223372036854775807", 10,     9223372036854775807ULL},
>                {"9223372036854775808", 10,     9223372036854775808ULL},
> @@ -369,12 +369,12 @@ static void __init test_kstrtos64_ok(void)
>                {"65537",       10,     65537},
>                {"2147483646",  10,     2147483646},
>                {"2147483647",  10,     2147483647},
> -               {"2147483648",  10,     2147483648},
> -               {"2147483649",  10,     2147483649},
> -               {"4294967294",  10,     4294967294},
> -               {"4294967295",  10,     4294967295},
> -               {"4294967296",  10,     4294967296},
> -               {"4294967297",  10,     4294967297},
> +               {"2147483648",  10,     2147483648LL},
> +               {"2147483649",  10,     2147483649LL},
> +               {"4294967294",  10,     4294967294LL},
> +               {"4294967295",  10,     4294967295LL},
> +               {"4294967296",  10,     4294967296LL},
> +               {"4294967297",  10,     4294967297LL},
>                {"9223372036854775806", 10,     9223372036854775806LL},
>                {"9223372036854775807", 10,     9223372036854775807LL},
>        };
> @@ -418,10 +418,10 @@ static void __init test_kstrtou32_ok(void)
>                {"65537",       10,     65537},
>                {"2147483646",  10,     2147483646},
>                {"2147483647",  10,     2147483647},
> -               {"2147483648",  10,     2147483648},
> -               {"2147483649",  10,     2147483649},
> -               {"4294967294",  10,     4294967294},
> -               {"4294967295",  10,     4294967295},
> +               {"2147483648",  10,     2147483648U},
> +               {"2147483649",  10,     2147483649U},
> +               {"4294967294",  10,     4294967294U},
> +               {"4294967295",  10,     4294967295U},
>        };
>        TEST_OK(kstrtou32, u32, "%u", test_u32_ok);
>  }

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
\
 
 \ /
  Last update: 2011-03-26 18:27    [W:0.276 / U:0.132 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site