brctl: tweak help text, fix comments
authorDenys Vlasenko <vda.linux@googlemail.com>
Sat, 12 Oct 2019 17:51:46 +0000 (19:51 +0200)
committerDenys Vlasenko <vda.linux@googlemail.com>
Sat, 12 Oct 2019 17:51:46 +0000 (19:51 +0200)
function                                             old     new   delta
packed_usage                                       33255   33236     -19

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
networking/brctl.c

index 09c55cd0e5e68a71e80011368c3905b241ec3b25..8ed9d2096ccd97cee45999aca2e7f09af112d205 100644 (file)
@@ -53,8 +53,9 @@
 //usage:     "\n       addif BRIDGE IFACE      Add IFACE to BRIDGE"
 //usage:     "\n       delif BRIDGE IFACE      Delete IFACE from BRIDGE"
 //usage:       IF_FEATURE_BRCTL_FANCY(
-//usage:     "\n       stp BRIDGE 1/yes/on|0/no/off    STP on/off"
+//usage:     "\n       showmacs BRIDGE                 List MAC addresses"
 //usage:     "\n       showstp BRIDGE                  Show STP info"
+//usage:     "\n       stp BRIDGE 1/yes/on|0/no/off    Set STP on/off"
 //usage:     "\n       setageing BRIDGE SECONDS        Set ageing time"
 //usage:     "\n       setfd BRIDGE SECONDS            Set bridge forward delay"
 //usage:     "\n       sethello BRIDGE SECONDS         Set hello time"
 //usage:     "\n       setbridgeprio BRIDGE PRIO       Set bridge priority"
 //usage:     "\n       setportprio BRIDGE IFACE PRIO   Set port priority"
 //usage:     "\n       setpathcost BRIDGE IFACE COST   Set path cost"
-//usage:     "\n       showmacs BRIDGE                 List MAC addresses"
 //usage:       )
 // Not yet implemented:
-//                     hairpin BRIDGE IFACE on|off     Hairpin on/off
+//                     hairpin BRIDGE IFACE on|off     Set hairpin on/off
 
 #include "libbb.h"
 #include "common_bufsiz.h"
@@ -566,7 +566,7 @@ int brctl_main(int argc UNUSED_PARAM, char **argv)
        }
 #endif
 
-       if (!*argv) /* all but 'show' need at least one argument */
+       if (!*argv) /* All of the below need at least one argument */
                bb_show_usage();
 
        br = *argv++;
@@ -599,7 +599,7 @@ int brctl_main(int argc UNUSED_PARAM, char **argv)
                return EXIT_SUCCESS;
        }
 
-       if (!*argv) /* all but 'addbr/delbr' need at least two arguments */
+       if (!*argv) /* All of the below need at least two arguments */
                bb_show_usage();
 
 #if ENABLE_FEATURE_BRCTL_FANCY