]> git.ipfire.org Git - thirdparty/util-linux.git/commitdiff
kill: flip all comparions to be in smaller - greater order
authorSami Kerola <kerolasa@iki.fi>
Sun, 9 Mar 2014 16:45:53 +0000 (11:45 -0500)
committerSami Kerola <kerolasa@iki.fi>
Sun, 30 Mar 2014 20:32:35 +0000 (21:32 +0100)
This makes code more readable.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
misc-utils/kill.c

index e7bf5f49a3830f5a7c310aac1946a2acf63177f2..877c24e17d2435066353ce7848bcbf5c41f794d9 100644 (file)
@@ -170,7 +170,7 @@ int main(int argc, char **argv)
        /* Loop through the arguments.  Actually, -a is the only option
         * can be used with other options.  The 'kill' is basically a
         * one-option-at-most program. */
-       for (argc--, argv++; argc > 0; argc--, argv++) {
+       for (argc--, argv++; 0 < argc; argc--, argv++) {
                arg = *argv;
                if (*arg != '-')
                        break;
@@ -195,7 +195,7 @@ int main(int argc, char **argv)
                                printsignals(stdout, 0);
                                return EXIT_SUCCESS;
                        }
-                       if (argc > 2)
+                       if (2 < argc)
                                return usage(EXIT_FAILURE);
                        /* argc == 2, accept "kill -l $?" */
                        arg = argv[1];
@@ -322,7 +322,7 @@ static int rtsig_to_signum(char *sig)
        if (!ep || sig == ep || errno || num < 0)
                return -1;
        num = maxi ? SIGRTMAX - num : SIGRTMIN + num;
-       if (num < SIGRTMIN || num > SIGRTMAX)
+       if (num < SIGRTMIN || SIGRTMAX < num)
                return -1;
        return num;
 }
@@ -354,9 +354,9 @@ static int arg_to_signum(char *arg, int maskbit)
 
        if (isdigit(*arg)) {
                numsig = strtol(arg, &ep, 10);
-               if (numsig >= NSIG && maskbit && (numsig & 128) != 0)
+               if (NSIG <= numsig && maskbit && (numsig & 128) != 0)
                        numsig -= 128;
-               if (*ep != 0 || numsig < 0 || numsig >= NSIG)
+               if (*ep != 0 || numsig < 0 || NSIG <= numsig)
                        return (-1);
                return (numsig);
        }
@@ -380,7 +380,7 @@ static void printsig(int sig)
                }
        }
 #ifdef SIGRTMIN
-       if (sig >= SIGRTMIN && sig <= SIGRTMAX) {
+       if (SIGRTMIN <= sig && sig <= SIGRTMAX) {
                printf("RT%d\n", sig - SIGRTMIN);
                return;
        }
@@ -407,7 +407,7 @@ static void printsignals(FILE *fp, int pretty)
        if (!pretty) {
                for (n = 0; n < ARRAY_SIZE(sys_signame); n++) {
                        lth = 1 + strlen(sys_signame[n].name);
-                       if (lpos + lth > 72) {
+                       if (72 < lpos + lth) {
                                fputc('\n', fp);
                                lpos = 0;
                        } else if (lpos)