strings --include-all-whitespace doesn't take an argument

Message ID e94c919c-df20-c5cc-4945-8aeebd4b9556@debian.org
State New
Headers show
Series
  • strings --include-all-whitespace doesn't take an argument
Related show

Commit Message

Matthias Klose Feb. 15, 2018, 1:37 a.m.
strings --include-all-whitespace doesn't take an argument, fix the behavior
according to the documentation.

binutils/

        * strings.c(long_options): include-all-whitespace does not take
        an extra agument.

Comments

Nick Clifton Feb. 16, 2018, 11:07 a.m. | #1
Hi Matthias,

> strings --include-all-whitespace doesn't take an argument, fix the behavior

> according to the documentation.

> 

> binutils/

> 

>         * strings.c(long_options): include-all-whitespace does not take

>         an extra agument.


Approved - please apply.  (This would count as an obvious fix).

Cheers
  Nick

Patch

binutils/

	* strings.c(long_options): include-all-whitespace does not take
	an extra agument.

--- a/binutils/strings.c
+++ b/binutils/strings.c
@@ -125,7 +125,7 @@  static struct option long_options[] =
   {"print-file-name", no_argument, NULL, 'f'},
   {"bytes", required_argument, NULL, 'n'},
   {"radix", required_argument, NULL, 't'},
-  {"include-all-whitespace", required_argument, NULL, 'w'},
+  {"include-all-whitespace", no_argument, NULL, 'w'},
   {"encoding", required_argument, NULL, 'e'},
   {"target", required_argument, NULL, 'T'},
   {"output-separator", required_argument, NULL, 's'},