On Tue, Apr 5, 2011 at 11:00 AM, Peter Tyser wrote:
>> +#ifdef CONFIG_CMD_GREPENV
>> +static int do_env_grep (cmd_tbl_t *cmdtp, int flag, int argc, char * const 
>> argv[])
>> +{
>> +     ENTRY *match;
>> +     int matched[env_htab.size];
>> +     int rcode = 1, idx;
>> +
>> +     if (argc < 2) {
>> +             cmd_usage(cmdtp);
>> +             return 1;
>> +     }
>
> This could be:
> if (argc < 2)
>        return cmd_usage(cmdtp);
s/could/should/
-mike
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to