Skip to content
Snippets Groups Projects
Commit ca6a0fee authored by Russ Cox's avatar Russ Cox
Browse files

more "declared and not used".

the last round omitted := range and only
checked 1 out of N vars in a multi-var :=

R=r
OCL=34624
CL=34638
parent 1a319890
No related branches found
No related tags found
No related merge requests found
Showing
with 35 additions and 32 deletions
......@@ -147,7 +147,7 @@ func TestPartialRead(t *testing.T) {
t.Fatalf("Didn't get first file: %v", err);
}
buf := make([]byte, 4);
if n, err := io.ReadFull(tr, buf); err != nil {
if _, err := io.ReadFull(tr, buf); err != nil {
t.Fatalf("Unexpected error: %v", err);
}
if expected := strings.Bytes("Kilt"); !bytes.Equal(buf, expected) {
......@@ -160,7 +160,7 @@ func TestPartialRead(t *testing.T) {
t.Fatalf("Didn't get second file: %v", err);
}
buf = make([]byte, 6);
if n, err := io.ReadFull(tr, buf); err != nil {
if _, err := io.ReadFull(tr, buf); err != nil {
t.Fatalf("Unexpected error: %v", err);
}
if expected := strings.Bytes("Google"); !bytes.Equal(buf, expected) {
......
......@@ -64,7 +64,7 @@ var writerTests = []*writerTest{
func bytestr(offset int, b []byte) string {
const rowLen = 32;
s := fmt.Sprintf("%04x ", offset);
for i, ch := range b {
for _, ch := range b {
switch {
case '0' <= ch && ch <= '9', 'A' <= ch && ch <= 'Z', 'a' <= ch && ch <= 'z':
s += fmt.Sprintf(" %c", ch);
......@@ -114,7 +114,7 @@ testLoop:
t.Errorf("test %d, entry %d: Failed writing header: %v", i, j, err);
continue testLoop
}
if n, err := io.WriteString(tw, entry.contents); err != nil {
if _, err := io.WriteString(tw, entry.contents); err != nil {
t.Errorf("test %d, entry %d: Failed writing contents: %v", i, j, err);
continue testLoop
}
......
......@@ -158,7 +158,7 @@ func TestDecodeCorrupt(t *testing.T) {
for _, e := range examples {
dbuf := make([]byte, StdEncoding.DecodedLen(len(e.e)));
count, err := StdEncoding.Decode(strings.Bytes(e.e), dbuf);
_, err := StdEncoding.Decode(strings.Bytes(e.e), dbuf);
switch err := err.(type) {
case CorruptInputError:
testEqual(t, "Corruption in %q at offset %v, want %v", e.e, int(err), e.p);
......
......@@ -591,7 +591,7 @@ func divmod(x, y []digit2) ([]digit2, []digit2) {
// If y == 0, a division-by-zero run-time error occurs.
//
func (x Natural) Div(y Natural) Natural {
q, r := divmod(unpack(x), unpack(y));
q, _ := divmod(unpack(x), unpack(y));
return pack(q);
}
......@@ -601,7 +601,7 @@ func (x Natural) Div(y Natural) Natural {
// If y == 0, a division-by-zero run-time error occurs.
//
func (x Natural) Mod(y Natural) Natural {
q, r := divmod(unpack(x), unpack(y));
_, r := divmod(unpack(x), unpack(y));
return pack(r);
}
......
......@@ -152,7 +152,7 @@ func TestNatConv(t *testing.T) {
test_msg = "NatConvG";
x := Nat(100);
y, b, _ := NatFromString(fmt.Sprintf("%b", &x), 2);
y, _, _ := NatFromString(fmt.Sprintf("%b", &x), 2);
nat_eq(100, y, x);
}
......
......@@ -156,7 +156,7 @@ func TestReader(t *testing.T) {
bufreader := bufreaders[j];
bufsize := bufsizes[k];
read := readmaker.fn(bytes.NewBuffer(textbytes));
buf, e := NewReaderSize(read, bufsize);
buf, _ := NewReaderSize(read, bufsize);
s := bufreader.fn(buf);
if s != text {
t.Errorf("reader=%s fn=%s bufsize=%d want=%q got=%q",
......@@ -193,7 +193,7 @@ func readRuneSegments(t *testing.T, segments []string) {
want := strings.Join(segments, "");
r := NewReader(&StringReader{data: segments});
for {
rune, size, err := r.ReadRune();
rune, _, err := r.ReadRune();
if err != nil {
if err != os.EOF {
return;
......@@ -293,9 +293,9 @@ var errorWriterTests = []errorWriterTest {
}
func TestWriteErrors(t *testing.T) {
for i, w := range errorWriterTests {
for _, w := range errorWriterTests {
buf := NewWriter(w);
n, e := buf.Write(strings.Bytes("hello world"));
_, e := buf.Write(strings.Bytes("hello world"));
if e != nil {
t.Errorf("Write hello to %v: %v", w, e);
continue;
......
......@@ -164,7 +164,7 @@ func TestMixedReadsAndWrites(t *testing.T) {
rlen := rand.Intn(len(data));
fub := make([]byte, rlen);
n, err := buf.Read(fub);
n, _ := buf.Read(fub);
s = s[n : len(s)];
}
empty(t, "TestMixedReadsAndWrites (2)", &buf, s, make([]byte, buf.Len()));
......
......@@ -190,8 +190,8 @@ func Map(mapping func(rune int) int, s []byte) []byte {
maxbytes := len(s); // length of b
nbytes := 0; // number of bytes encoded in b
b := make([]byte, maxbytes);
for wid, i := 0, 0; i < len(s); i += wid {
wid = 1;
for i := 0; i < len(s); {
wid := 1;
rune := int(s[i]);
if rune < utf8.RuneSelf {
rune = mapping(rune);
......@@ -209,6 +209,7 @@ func Map(mapping func(rune int) int, s []byte) []byte {
b = nb;
}
nbytes += utf8.EncodeRune(rune, b[nbytes:maxbytes]);
i += wid;
}
return b[0:nbytes];
}
......@@ -232,8 +233,8 @@ func Title(s []byte) []byte {
// removed, as defined by Unicode.
func TrimSpace(s []byte) []byte {
start, end := 0, len(s);
for wid := 0; start < end; start += wid {
wid = 1;
for start < end {
wid := 1;
rune := int(s[start]);
if rune >= utf8.RuneSelf {
rune, wid = utf8.DecodeRune(s[start:end])
......@@ -241,9 +242,10 @@ func TrimSpace(s []byte) []byte {
if !unicode.IsSpace(rune) {
break;
}
start += wid;
}
for wid := 0; start < end; end -= wid {
wid = 1;
for start < end {
wid := 1;
rune := int(s[end-1]);
if rune >= utf8.RuneSelf {
// Back up carefully looking for beginning of rune. Mustn't pass start.
......@@ -257,6 +259,7 @@ func TrimSpace(s []byte) []byte {
if !unicode.IsSpace(rune) {
break;
}
end -= wid;
}
return s[start:end];
}
......
......@@ -216,7 +216,7 @@ func Bytes(s string) []byte {
// Execute f on each test case. funcName should be the name of f; it's used
// in failure reports.
func runStringTests(t *testing.T, f func([]byte) []byte, funcName string, testCases []StringTest) {
for i, tc := range testCases {
for _, tc := range testCases {
actual := string(f(Bytes(tc.in)));
if actual != tc.out {
t.Errorf("%s(%q) = %q; want %q", funcName, tc.in, actual, tc.out);
......@@ -275,7 +275,7 @@ var addtests = []AddTest {
}
func TestAdd(t *testing.T) {
for i, test := range addtests {
for _, test := range addtests {
b := make([]byte, len(test.s), test.cap);
for i := 0; i < len(test.s); i++ {
b[i] = test.s[i]
......
......@@ -121,7 +121,7 @@ func (h *huffmanDecoder) init(bits []int) bool {
// compute min and max length.
var count [maxCodeLen+1]int;
var min, max int;
for i, n := range bits {
for _, n := range bits {
if n == 0 {
continue;
}
......
......@@ -116,7 +116,7 @@ func (z *Inflater) read2() (uint32, os.Error) {
}
func (z *Inflater) readHeader(save bool) os.Error {
n, err := io.ReadFull(z.r, z.buf[0:10]);
_, err := io.ReadFull(z.r, z.buf[0:10]);
if err != nil {
return err;
}
......
......@@ -282,7 +282,7 @@ var gzipTests = []gzipTest {
func TestInflater(t *testing.T) {
b := new(bytes.Buffer);
for i, tt := range gzipTests {
for _, tt := range gzipTests {
in := bytes.NewBuffer(tt.gzip);
gzip, err := NewInflater(in);
if err != nil {
......
......@@ -39,7 +39,7 @@ func NewInflater(r io.Reader) (io.ReadCloser, os.Error) {
} else {
z.r = bufio.NewReader(r);
}
n, err := io.ReadFull(z.r, z.scratch[0:2]);
_, err := io.ReadFull(z.r, z.scratch[0:2]);
if err != nil {
return nil, err;
}
......
......@@ -77,7 +77,7 @@ var zlibTests = []zlibTest {
func TestInflater(t *testing.T) {
b := new(bytes.Buffer);
for i, tt := range zlibTests {
for _, tt := range zlibTests {
in := bytes.NewBuffer(tt.compressed);
zlib, err := NewInflater(in);
if err != nil {
......
......@@ -102,7 +102,7 @@ func (p *IntVector) Less(i, j int) bool {
// Iterate over all elements; driver for range
func (p *IntVector) iterate(c chan<- int) {
for i, v := range p.a {
for _, v := range p.a {
c <- v.(int)
}
close(c);
......
......@@ -101,7 +101,7 @@ func (p *StringVector) Less(i, j int) bool {
// Iterate over all elements; driver for range
func (p *StringVector) iterate(c chan<- string) {
for i, v := range p.a {
for _, v := range p.a {
c <- v.(string)
}
close(c);
......
......@@ -230,7 +230,7 @@ func (p *Vector) Swap(i, j int) {
// Iterate over all elements; driver for range
func (p *Vector) iterate(c chan<- Element) {
for i, v := range p.a {
for _, v := range p.a {
c <- v
}
close(c);
......
......@@ -67,7 +67,7 @@ var cbcAESTests = []cbcTest {
}
func TestCBC_AES(t *testing.T) {
for i, tt := range cbcAESTests {
for _, tt := range cbcAESTests {
test := tt.name;
c, err := aes.NewCipher(tt.key);
......
......@@ -271,7 +271,7 @@ var cfbAESTests = []cfbTest {
}
func TestCFB_AES(t *testing.T) {
for i, tt := range cfbAESTests {
for _, tt := range cfbAESTests {
test := tt.name;
if tt.s == 1 {
......
......@@ -70,7 +70,7 @@ func (d *cmac) Reset() {
// Write adds the given data to the digest state.
func (d *cmac) Write(p []byte) (n int, err os.Error) {
// Xor input into ci.
for i, c := range p {
for _, c := range p {
// If ci is full, encrypt and start over.
if d.p >= len(d.ci) {
d.c.Encrypt(d.ci, d.ci);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment