[PATCH 2/2] menu: add support for disable entry

Jean-Christophe PLAGNIOL-VILLARD plagnioj at jcrosoft.com
Tue May 29 06:35:07 EDT 2012


On 10:35 Tue 29 May     , Sascha Hauer wrote:
> On Sun, May 27, 2012 at 03:57:13PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj at jcrosoft.com>
> > ---
> >  .../arm/boards/at91sam9m10g45ek/env/bin/boot_board |    2 +
> >  commands/menu.c                                    |   15 +++++-
> >  common/menu.c                                      |   44 +++++++++++++------
> >  include/menu.h                                     |    1 +
> >  4 files changed, 45 insertions(+), 17 deletions(-)
> > 
> > @@ -450,6 +458,7 @@ static const __maybe_unused char cmd_menu_help[] =
> >  "Add an entry\n"
> >  "  (-R for do no exit the menu after executing the command)\n"
> >  "  (-b for box style 1 for selected)\n"
> > +"  (-D for disabel style command is not needed)\n"
> 
> s/disabel/disable/
> 
> Better talk about comment lines instead? If I understand this patch
> correctly that's what it does.
yes and no as I get in mind to beable to change the type of the menu_entry at
runtime

Best Regards,
J.



More information about the barebox mailing list