diff options
| author | Daniel Martí <mvdan@mvdan.cc> | 2017-08-21 22:23:14 +0200 |
|---|---|---|
| committer | Daniel Martí <mvdan@mvdan.cc> | 2017-08-29 20:57:41 +0000 |
| commit | fbc8973a6bc88b50509ea738f475b36ef756bf90 (patch) | |
| tree | bec33b02e805593213b72d9908443131af909fe0 /src/runtime/cgocall.go | |
| parent | 305fd9179d2199cd1bb64402405857d2f2d02478 (diff) | |
| download | go-fbc8973a6bc88b50509ea738f475b36ef756bf90.tar.gz go-fbc8973a6bc88b50509ea738f475b36ef756bf90.zip | |
all: join some chained ifs to unindent code
Found with mvdan.cc/unindent. It skipped the cases where parentheses
would need to be added, where comments would have to be moved elsewhere,
or where actions and simple logic would mix.
One of them was of the form "err != nil && err == io.EOF", so the first
part was removed.
Change-Id: Ie504c2b03a2c87d10ecbca1b9270069be1171b91
Reviewed-on: https://go-review.googlesource.com/57690
Run-TryBot: Daniel Martí <mvdan@mvdan.cc>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Diffstat (limited to 'src/runtime/cgocall.go')
| -rw-r--r-- | src/runtime/cgocall.go | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/runtime/cgocall.go b/src/runtime/cgocall.go index 755269ebd2..ce4d707e06 100644 --- a/src/runtime/cgocall.go +++ b/src/runtime/cgocall.go @@ -580,10 +580,8 @@ func cgoCheckUnknownPointer(p unsafe.Pointer, msg string) (base, i uintptr) { // No more possible pointers. break } - if hbits.isPointer() { - if cgoIsGoPointer(*(*unsafe.Pointer)(unsafe.Pointer(base + i))) { - panic(errorString(msg)) - } + if hbits.isPointer() && cgoIsGoPointer(*(*unsafe.Pointer)(unsafe.Pointer(base + i))) { + panic(errorString(msg)) } hbits = hbits.next() } |