Re: [PATCH 1/5] kconfig: make hexadecimal prefix mandatory for S_HEX default value

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Sun, Dec 5, 2010 at 1:29 AM, Arnaud Lacombe <lacombar@xxxxxxxxx> wrote:
> Signed-off-by: Arnaud Lacombe <lacombar@xxxxxxxxx>
> ---
>  scripts/kconfig/symbol.c |    6 ++++--
>  1 files changed, 4 insertions(+), 2 deletions(-)
>
ping ?

[I do not really care about that one, but still... The next patch does
not address ~100 other warnings potentially triggered in arch/
Kconfig.]

 - Arnaud


> diff --git a/scripts/kconfig/symbol.c b/scripts/kconfig/symbol.c
> index af6e9f3..ad7dbe7 100644
> --- a/scripts/kconfig/symbol.c
> +++ b/scripts/kconfig/symbol.c
> @@ -548,8 +548,10 @@ bool sym_string_valid(struct symbol *sym, const char *str)
>                }
>                return true;
>        case S_HEX:
> -               if (str[0] == '0' && (str[1] == 'x' || str[1] == 'X'))
> -                       str += 2;
> +               if (str[0] != '\0' &&
> +                   !(str[0] == '0' && (str[1] == 'x' || str[1] == 'X')))
> +                       return false;
> +               str += 2;
>                ch = *str++;
>                do {
>                        if (!isxdigit(ch))
> --
> 1.7.2.30.gc37d7.dirty
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux&nblp;USB Development]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite Secrets]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux