diff options
author | Carsten Otte <cotte@de.ibm.com> | 2006-03-25 03:08:22 -0800 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-03-25 08:23:01 -0800 |
commit | 55148548124e3e52e8921f1cb0e325111ef9cbb1 (patch) | |
tree | b1a84317305d060b38bc6fcb1ae4006a76f22071 /fs/binfmt_elf.c | |
parent | 06842415149371127b91d15774cff95dcccc1845 (diff) | |
download | lwn-55148548124e3e52e8921f1cb0e325111ef9cbb1.tar.gz lwn-55148548124e3e52e8921f1cb0e325111ef9cbb1.zip |
[PATCH] remove needless check in binfmt_elf.c
Local variable i is unsigned int and thus cannot be negative.
(akpm: unsigneds shouldn't be called `i'. This value cannot possibly be
negative anyway).
Signed-off-by: Carsten Otte <cotte@de.ibm.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'fs/binfmt_elf.c')
-rw-r--r-- | fs/binfmt_elf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/binfmt_elf.c b/fs/binfmt_elf.c index 61c21e7dc95d..4349113881fb 100644 --- a/fs/binfmt_elf.c +++ b/fs/binfmt_elf.c @@ -1334,7 +1334,7 @@ static int fill_psinfo(struct elf_prpsinfo *psinfo, struct task_struct *p, i = p->state ? ffz(~p->state) + 1 : 0; psinfo->pr_state = i; - psinfo->pr_sname = (i < 0 || i > 5) ? '.' : "RSDTZW"[i]; + psinfo->pr_sname = (i > 5) ? '.' : "RSDTZW"[i]; psinfo->pr_zomb = psinfo->pr_sname == 'Z'; psinfo->pr_nice = task_nice(p); psinfo->pr_flag = p->flags; |