On Tue, 2008-08-12 at 03:36 +0200, Denys Vlasenko wrote:
> On Thursday 07 August 2008 23:12, Wade Berrier wrote:
> 
> I propose to write this more compactly like this:
> 
> 
>                                 if (varvalue) {
> #if ENABLE_FEATURE_IFUPDOWN_IP
>                                         /* "hwaddress <class> <address>":
>                                          * unlike ifconfig, ip doesnt want 
> <class>
>                                          * (usually "ether" keyword). Skip 
> it. */
>                                         if (strncmp(command, "hwaddress", 9) 
> == 0) {
>                                                 varvalue = 
> skip_whitespace(skip_non_whitespace(varvalue));
>                                         }
> #endif
>                                         addstr(&result, varvalue, 
> strlen(varvalue));
> 
> 
> I applied both your patches wit hsmall edits. Thanks!

Ah, I should have known there were functions to do that already :)  

Thanks for committing,

Wade

_______________________________________________
busybox mailing list
[email protected]
http://busybox.net/cgi-bin/mailman/listinfo/busybox

Reply via email to