path: root/
diff options
authorBen Hutchings <>2011-10-29 01:15:34 +0100
committerBen Hutchings <>2011-11-03 19:13:04 +0000
commit127f80691f96b573db633c7dfe9fc36006dd6979 (patch)
tree5268575dc5c0e369c6aa1021453d570ff61e6f73 /
parent75300bbbcb20650297d0c29b945167a1e3347471 (diff)
Move argument parsing to sub-command functions
Instead of doing most of the argument parsing in parse_cmdline(), find the sub-command and (if required) the device name in main() and do the rest in the sub-command handler function. Pass argc and argp around in struct cmd_context. This also tightens up argument parsing slightly: extraneous or missing arguments will now result in an error in a few cases where they were previously ignored. All test cases now pass. Replace sub-command dispatch in doit() with a function pointer in struct option. Signed-off-by: Ben Hutchings <>
Diffstat (limited to '')
1 files changed, 2 insertions, 2 deletions
diff --git a/ b/
index 3304fe5..7c39629 100644
--- a/
+++ b/
@@ -252,11 +252,11 @@ ethtool \- query or control network driver and hardware settings
.B ethtool \-n
.I devname
-.RB [ rx\-flow\-hash \ \*(FL]
+.BR rx\-flow\-hash \ \*(FL
.B ethtool \-N
.I devname
-.RB [ rx\-flow\-hash \ \*(FL \: \*(HO]
+.BR rx\-flow\-hash \ \*(FL \: \*(HO
.B ethtool \-w|\-\-get\-dump
.I devname