All of lore.kernel.org
 help / color / mirror / Atom feed
From: Adrian Bunk <bunk@stusta.de>
To: Roman Zippel <zippel@linux-m68k.org>
Cc: Andrew Morton <akpm@osdl.org>,
	kbuild-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org,
	Jesper Juhl <juhl-lkml@dif.dk>
Subject: Re: [2.6 patch] Kconfig: rename "---help---" to "help" in Kconfig files (first part)
Date: Tue, 3 May 2005 11:22:03 +0200	[thread overview]
Message-ID: <20050503092202.GC3592@stusta.de> (raw)
In-Reply-To: <Pine.LNX.4.61.0505031107120.996@scrub.home>

On Tue, May 03, 2005 at 11:10:48AM +0200, Roman Zippel wrote:
> Hi,
> 
> On Tue, 3 May 2005, Adrian Bunk wrote:
> 
> > This patch is the majority of a patch by Jesper Juhl.
> > 
> > This patch renames all instances of "---help---" to simply "help" in all 
> > of the Kconfig files.
> > 
> > The main reason for this patch (quoting Jesper) is:
> > 
> > Consistency. out of ~4000 help entries in 134 Kconfig files, 747 of 
> > those entries use "---help---" as the keyword, the rest use just "help". 
> > So the users of "---help---" are clearly a minority and by renaming them 
> > we make things consistent. - I hate inconsistency. :-)
> 
> This has nothing to do with consistency but with readability.
> This was introduced to better separate the help in large menu entries. In 
> order to accept this patch, I would either like hear reasons, why this 
> isn't needed anymore or I'd like to see an alternative, more consistent 
> separator.

The separator used for the help is to indent help texts by two 
additional spaces.

IMHO, Kconfig files are quite readable due to this indentation even 
though only a minority of the entries was using "---help---" even 
before this patch.

> bye, Roman

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed


  reply	other threads:[~2005-05-03  9:22 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-05-03  0:34 [2.6 patch] Kconfig: rename "---help---" to "help" in Kconfig files (first part) Adrian Bunk
2005-05-03  9:10 ` Roman Zippel
2005-05-03  9:22   ` Adrian Bunk [this message]
2005-05-03 10:09     ` Roman Zippel
2005-05-03 10:57       ` Adrian Bunk
2005-05-03 13:47         ` Roman Zippel
2005-05-04 11:50           ` Adrian Bunk
     [not found] <3ZTLT-3D3-1@gated-at.bofh.it>
     [not found] ` <401Js-1AH-1@gated-at.bofh.it>
     [not found]   ` <401Tf-1Hp-9@gated-at.bofh.it>
     [not found]     ` <402FB-2mN-15@gated-at.bofh.it>
2005-05-03 13:56       ` Bodo Eggert <harvested.in.lkml@posting.7eggert.dyndns.org>

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=20050503092202.GC3592@stusta.de \
    --to=bunk@stusta.de \
    --cc=akpm@osdl.org \
    --cc=juhl-lkml@dif.dk \
    --cc=kbuild-devel@lists.sourceforge.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=zippel@linux-m68k.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.