diff options
| author | Tobias Klauser <tklauser@distanz.ch> | 2017-11-29 11:46:48 +0100 |
|---|---|---|
| committer | Tobias Klauser <tobias.klauser@gmail.com> | 2017-11-29 15:53:14 +0000 |
| commit | 06d87bf854ff61c2ea72dec6fc10c4c3fd59a76b (patch) | |
| tree | 520ebad57ced934379a5d4bc3c8bd5b7decc5478 | |
| parent | c3aeeef584abb98e9d442193ab37bc89a5f5300b (diff) | |
| download | go-06d87bf854ff61c2ea72dec6fc10c4c3fd59a76b.tar.xz | |
os: correct err check in TestChdirAndGetwd
Due to err being shadowed in the else brach, the actual err return of
fd1.Chdir() is never checked. Fix it by not shadowing err anymore.
Change-Id: I9f1d52e88d8bc9a1c035960aa7af9f5224a63ab0
Reviewed-on: https://go-review.googlesource.com/80556
Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
| -rw-r--r-- | src/os/os_test.go | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/os/os_test.go b/src/os/os_test.go index eb8a7d1b92..2d608f4b24 100644 --- a/src/os/os_test.go +++ b/src/os/os_test.go @@ -1239,9 +1239,9 @@ func TestChdirAndGetwd(t *testing.T) { if mode == 0 { err = Chdir(d) } else { - fd1, err := Open(d) - if err != nil { - t.Errorf("Open %s: %s", d, err) + fd1, err1 := Open(d) + if err1 != nil { + t.Errorf("Open %s: %s", d, err1) continue } err = fd1.Chdir() |