Re: [RFC] kconfig: zconfdump() clean-up

[Michal Marek <mmarek@suse.cz>]

On 16.5.2011 06:06, Arnaud Lacombe wrote:
> Hi Michal,
>
> The following patch is a summarize of changes I made in zconfdump() last week
> when debugging Yann's multiple choice issue. This might help debugging a bit,
> espectially by exposing the tree depth.
>
> Considering the following Kconfig:
>
> config A
>     bool "A"
> if A
> choice C
>     bool "C"
> config C1
>     bool "C1"
> config C2
>     bool "C2"
> endchoice
> endif # A
>
> config B
>     bool "B"
>     depends on ! A
> if B
> choice C
>     bool "C"
> config C1
>     bool "C1"
> config C2
>     bool "C2"
> endchoice
> endif # A
>
> It generates the following output, after menu_finalize():
>
> config A
>    boolean
>    symbol
>    prompt "A"
>
>          choice C
>            boolean
>            symbol
>            #choice
>            prompt "C" if A

Why not. zconfdump() is only used for debugging and you've been doing 
most of the kconfig work lately, so if this suits you, then I'll merge 
it. I only spotted one minor issue:

commit 21685d396a123a576bbe487651cb5cc04fa5f9a3
Author: Arnaud Lacombe <lacombar@gmail.com>
Date:   Tue May 10 22:55:20 2011 -0400

     kconfig/zconfdump(): output newline at the *end* of statement

puts the newline between the symbol statement and it's help text (if 
any). So if there is help text defined, it will appear attached to the 
next symbol. But it's not really an issue.

Michal