/* 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;
printsignals(stdout, 0);
return EXIT_SUCCESS;
}
- if (argc > 2)
+ if (2 < argc)
return usage(EXIT_FAILURE);
/* argc == 2, accept "kill -l $?" */
arg = argv[1];
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;
}
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);
}
}
}
#ifdef SIGRTMIN
- if (sig >= SIGRTMIN && sig <= SIGRTMAX) {
+ if (SIGRTMIN <= sig && sig <= SIGRTMAX) {
printf("RT%d\n", sig - SIGRTMIN);
return;
}
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)