diff options
| author | Dave Cheney <dave@cheney.net> | 2016-04-06 18:43:23 +1000 |
|---|---|---|
| committer | Dave Cheney <dave@cheney.net> | 2016-04-06 21:04:43 +0000 |
| commit | 0c81248bf46f611b56e3ab38b4d83e449b3c8636 (patch) | |
| tree | 5ecc59d0b4a326a4c856b89840feafdd9f49eddd /src/runtime/lfstack.go | |
| parent | 04945edd40fff4d66321a4f98c1bb070b6356008 (diff) | |
| download | go-0c81248bf46f611b56e3ab38b4d83e449b3c8636.tar.xz | |
runtime: remove unused return value from lfstackUnpack
None of the two places that call lfstackUnpack use the second argument.
This simplifies a followup CL that merges the lfstack{Pack,Unpack}
implementations.
Change-Id: I3c93f6259da99e113d94f8c8027584da79c1ac2c
Reviewed-on: https://go-review.googlesource.com/21595
Run-TryBot: Dave Cheney <dave@cheney.net>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Diffstat (limited to 'src/runtime/lfstack.go')
| -rw-r--r-- | src/runtime/lfstack.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/runtime/lfstack.go b/src/runtime/lfstack.go index ea640eb12f..1261f54d97 100644 --- a/src/runtime/lfstack.go +++ b/src/runtime/lfstack.go @@ -15,7 +15,7 @@ import ( func lfstackpush(head *uint64, node *lfnode) { node.pushcnt++ new := lfstackPack(node, node.pushcnt) - if node1, _ := lfstackUnpack(new); node1 != node { + if node1 := lfstackUnpack(new); node1 != node { print("runtime: lfstackpush invalid packing: node=", node, " cnt=", hex(node.pushcnt), " packed=", hex(new), " -> node=", node1, "\n") throw("lfstackpush") } @@ -34,7 +34,7 @@ func lfstackpop(head *uint64) unsafe.Pointer { if old == 0 { return nil } - node, _ := lfstackUnpack(old) + node := lfstackUnpack(old) next := atomic.Load64(&node.next) if atomic.Cas64(head, old, next) { return unsafe.Pointer(node) |