]> code.delx.au - pulseaudio/commitdiff
pactl: Stop parsing option when the first non-option is encountered
authorPeter Meerwald <p.meerwald@bct-electronic.com>
Tue, 15 Apr 2014 08:58:21 +0000 (10:58 +0200)
committerPeter Meerwald <pmeerw@pmeerw.net>
Tue, 15 Apr 2014 21:53:13 +0000 (23:53 +0200)
fix bug
https://bugs.freedesktop.org/show_bug.cgi?id=77108

see getopt(3):
""By default, getopt() permutes the contents of argv as it scans, so that
  eventually all the nonoptions are at the end.  Two other modes are also
  implemented.   If  the first character of optstring is '+' or the envi‐
  ronment variable POSIXLY_CORRECT is set, then option  processing  stops
  as soon as a nonoption argument is encountered.  If the first character
  of optstring is '-', then each nonoption argv-element is handled as  if
  it were the argument of an option with character code 1.  (This is used
  by programs that were written to expect options and other argv-elements
  in any order and that care about the ordering of the two.)  The special
  argument "--" forces an end of option-scanning regardless of the  scan‐
  ning mode.""

prepend optstring with '+' to use POSIXLY_CORRECT mode

Signed-off-by: Peter Meerwald <pmeerw@pmeerw.net>
src/utils/pactl.c

index 8483464f50125dc0233b829c76f07363cf7583b4..555cb48101969e4e4c741c9599f32ca1652d5c20 100644 (file)
@@ -1596,7 +1596,7 @@ int main(int argc, char *argv[]) {
 
     proplist = pa_proplist_new();
 
-    while ((c = getopt_long(argc, argv, "s:n:h", long_options, NULL)) != -1) {
+    while ((c = getopt_long(argc, argv, "+s:n:h", long_options, NULL)) != -1) {
         switch (c) {
             case 'h' :
                 help(bn);