]> granicus.if.org Git - procps-ng/commitdiff
top: fixing command line parsing errors is now a habit
authorJim Warner <james.warner@comcast.net>
Wed, 21 Jun 2017 01:20:20 +0000 (20:20 -0500)
committerCraig Small <csmall@enc.com.au>
Tue, 4 Jul 2017 10:29:26 +0000 (20:29 +1000)
Ok, I admit it. I'm now tired of cleaning up after me.

This is the 3rd related tweak after that '-1' argument
was originally introduced. And with this patch we will
once again properly honor the '-o' and '-u|U' switches
without a need to be followed by an additional switch.

[ one can follow my unfortunate trail of alterations ]
[ beginning with my most recent fix referenced below ]

Reference(s):
commit 4b44aebd807bd132c14df20eabff2f2e9628ef97

Signed-off-by: Jim Warner <james.warner@comcast.net>
top/top.c

index 8c3bd05a6d9212c43576c3814cd41361aa36215b..4e81e66969e57ba31b3c9ca7f7f64046f3aeb768 100644 (file)
--- a/top/top.c
+++ b/top/top.c
@@ -3754,7 +3754,7 @@ static void parse_args (char **args) {
          // advance cp and jump over any numerical args used above
          if (*cp) cp += strspn(&cp[1], numbs_str);
 bump_cp:
-         ++cp;
+         if (*cp) ++cp;
       } // end: while (*cp)
    } // end: while (*args)