[PATCH 1/5] commands: nv: fail with verbose message if invoked without args
Enrico Jörns
ejo at pengutronix.de
Tue Oct 31 14:47:43 PDT 2017
Hi Lengfeld,
Am 31.10.2017 um 18:51 schrieb Stefan Lengfeld:
> Hi Jorns,
>
> On Mon, Oct 30, 2017 at 11:34:17AM +0100, Enrico Jorns wrote:
>> Signed-off-by: Enrico Jorns <ejo at pengutronix.de>
>> ---
>> commands/nv.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/commands/nv.c b/commands/nv.c
>> index 37cdb96647..1c5514dd32 100644
>> --- a/commands/nv.c
>> +++ b/commands/nv.c
>> @@ -56,6 +56,7 @@ static int do_nv(int argc, char *argv[])
>> argv += optind;
>>
>> if (argc < 1)
>> + printf("Invalid usage: Missing argument");
>> return COMMAND_ERROR_USAGE;
>
> Missing curly brackets for if clause. Indentation does not matter in C
> as it does in for example python. Now the function returns always
> 'COMMAND_ERROR_USAGE'.
ouh, you are right, must have lost them while merging the patches too
quickly.
But I tried my best to hide it, didn't I?
Best regards, Enrico
--
Pengutronix e.K. | Enrico Jörns |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-5080 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
More information about the barebox
mailing list