From: Michal Marek <mmarek@suse.cz>
To: Jan Beulich <JBeulich@suse.com>
Cc: linux-kbuild@vger.kernel.org, akpm@linux-foundation.org,
pebolle@tiscali.nl
Subject: Re: [PATCH v3 3/3] kconfig: allow use of relations other than (in)equality
Date: Thu, 11 Jun 2015 14:44:32 +0200 [thread overview]
Message-ID: <557982B0.2070504@suse.cz> (raw)
In-Reply-To: <54BE5DD10200007800057065@mail.emea.novell.com>
On 2015-01-20 13:53, Jan Beulich wrote:
> Over the years I found it desirable to be able to use all sorts of
> relations, not just (in)equality. And apparently I'm not the only one,
> as there's at least one example in the tree where the programmer
> assumed this would work (see DEBUG_UART_8250_WORD in
> arch/arm/Kconfig.debug). Another possible use would e.g. be to fold the
> two SMP/NR_CPUS prompts into one: SMP could be promptless, simply
> depending on NR_CPUS > 1.
>
> A (desirable) side effect of this change - resulting from numeric
> values now necessarily being compared as numbers rather than as
> strings - is that comparing hex values now works as expected: Other
> than int ones (which aren't allowed to have leading zeroes), zeroes
> following the 0x prefix made them compare unequal even if their values
> were equal.
>
> Question: Should "<>" and/or "==" then perhaps also be permitted?
>
> Signed-off-by: Jan Beulich <jbeulich@suse.com>
> ---
> v2: Drop stray debugging printf()s.
> ---
> scripts/kconfig/expr.c | 167 ++++++++++-
> scripts/kconfig/expr.h | 4
> scripts/kconfig/symbol.c | 4
> scripts/kconfig/zconf.l | 4
> scripts/kconfig/zconf.lex.c_shipped | 291 +++++++++++--------
> scripts/kconfig/zconf.tab.c_shipped | 524 +++++++++++++++++++-----------------
> scripts/kconfig/zconf.y | 9
Again, sorry for the extremely late reply.
Please submit the update of the _shipped files as a another patch,
referencing this one. Having ~800 generated edits among original changes
makes the whole patch hard to read, both for me and for anyone browsing
the git history in the future.
Michal
prev parent reply other threads:[~2015-06-11 12:44 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-20 12:53 [PATCH v3 3/3] kconfig: allow use of relations other than (in)equality Jan Beulich
2015-06-11 12:44 ` Michal Marek [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=557982B0.2070504@suse.cz \
--to=mmarek@suse.cz \
--cc=JBeulich@suse.com \
--cc=akpm@linux-foundation.org \
--cc=linux-kbuild@vger.kernel.org \
--cc=pebolle@tiscali.nl \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).