diff options
author | cperciva <cperciva@FreeBSD.org> | 2008-08-04 06:39:52 +0000 |
---|---|---|
committer | cperciva <cperciva@FreeBSD.org> | 2008-08-04 06:39:52 +0000 |
commit | 55239b4ba74bfd89b442877e0a7eb19b238fada4 (patch) | |
tree | 2ac1f33f454d0f949f4b8e9718c2b79c2eef55f4 | |
parent | bfce43df588d6c27e2859a749c641a25969021b6 (diff) | |
download | FreeBSD-src-55239b4ba74bfd89b442877e0a7eb19b238fada4.zip FreeBSD-src-55239b4ba74bfd89b442877e0a7eb19b238fada4.tar.gz |
Setting a variable to the same value twice doesn't actually make it
more likely to have the right value. Remove superfluous assignments.
Found by: LLVM/Clang Static Checker
-rw-r--r-- | sys/boot/common/misc.c | 2 | ||||
-rw-r--r-- | usr.bin/rs/rs.c | 1 | ||||
-rw-r--r-- | usr.bin/yacc/closure.c | 2 |
3 files changed, 1 insertions, 4 deletions
diff --git a/sys/boot/common/misc.c b/sys/boot/common/misc.c index f1a0491..c4c36ea 100644 --- a/sys/boot/common/misc.c +++ b/sys/boot/common/misc.c @@ -42,7 +42,7 @@ unargv(int argc, char *argv[]) int i; char *cp; - for (hlong = 0, i = 0, hlong = 0; i < argc; i++) + for (i = 0, hlong = 0; i < argc; i++) hlong += strlen(argv[i]) + 2; if(hlong == 0) diff --git a/usr.bin/rs/rs.c b/usr.bin/rs/rs.c index df01c57..82aaa9a 100644 --- a/usr.bin/rs/rs.c +++ b/usr.bin/rs/rs.c @@ -153,7 +153,6 @@ getfile(void) p++; } ep = getptrs(elem); - p = curline; do { if (flags & ONEPERLINE) { *ep = curline; diff --git a/usr.bin/yacc/closure.c b/usr.bin/yacc/closure.c index 24b1610..2124417 100644 --- a/usr.bin/yacc/closure.c +++ b/usr.bin/yacc/closure.c @@ -142,7 +142,6 @@ set_first_derives() } } - vrow += varsetsize; rrow += rulesetsize; } @@ -173,7 +172,6 @@ int n; int itemno; rulesetsize = WORDSIZE(nrules); - rsp = ruleset; rsend = ruleset + rulesetsize; for (rsp = ruleset; rsp < rsend; rsp++) *rsp = 0; |