[U-Boot] [PATCH v2 4/9] Stop using builtin_run_command()
Simon Glass
sjg at chromium.org
Fri Jan 13 22:33:42 CET 2012
Hi Wolfgang,
On Fri, Jan 13, 2012 at 1:25 PM, Wolfgang Denk <wd at denx.de> wrote:
> Dear Simon Glass,
>
> In message <CAPnjgZ1=1+xjHoAbJd4WETjC6Bj+6NPgGsXbStoGadieznF1=Q at mail.gmail.com> you wrote:
>>
>> > run_command2() in "common/main.c" (now renamed into run_command() will
>> > return 0 or 1 when used with the hush shell; it will never return < 0.
>>
>> Thanks for looking at this. I will see what I can figure out.
>
> I think there is some pre-existing inconsistency here, which pops up
> when you try to unify the code.
>
> I think we should implement standard shell handline here: commands
> return EXIT_SUCCESS or EXIT_FAILURE.
My hope was that I could return an additional code to indicate that
cmd_usage() should be called. This helps with code size and tidiness.
Do you think it is possible at all?
Regards,
Simon
>
> Best regards,
>
> Wolfgang Denk
>
> --
> DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
> The most exciting phrase to hear in science, the one that heralds new
> discoveries, is not "Eureka!" (I found it!) but "That's funny ..."
> -- Isaac Asimov
More information about the U-Boot
mailing list