]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/compile: remove redundant parameter from finishcompare
authorJosh Bleecher Snyder <josharian@gmail.com>
Wed, 23 Mar 2016 23:43:17 +0000 (16:43 -0700)
committerJosh Bleecher Snyder <josharian@gmail.com>
Thu, 24 Mar 2016 01:17:47 +0000 (01:17 +0000)
This is follow-up 3 of 3 to CL 20959.

Passes toolstash -cmp.

Change-Id: I06efded21bbc970cbefa10e8f2cac1ebc6942e1b
Reviewed-on: https://go-review.googlesource.com/21054
Run-TryBot: Josh Bleecher Snyder <josharian@gmail.com>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>

src/cmd/compile/internal/gc/walk.go

index ded7637eb2c5d0869fec8ab59e1f5a8021800faa..9862c24d4ff70dcf44f11325935fdef6a797e266 100644 (file)
@@ -3130,7 +3130,7 @@ func walkcompare(n *Node, init *Nodes) *Node {
                        r = Nod(OOROR, Nod(ONOT, ok, nil), Nod(ONE, x, r))
                }
                init.Append(expr)
-               n = finishcompare(n, n, r, init)
+               n = finishcompare(n, r, init)
                return n
        }
 
@@ -3201,7 +3201,7 @@ func walkcompare(n *Node, init *Nodes) *Node {
                if expr == nil {
                        expr = Nodbool(n.Op == OEQ)
                }
-               n = finishcompare(n, n, expr, init)
+               n = finishcompare(n, expr, init)
                return n
        }
 
@@ -3209,13 +3209,13 @@ func walkcompare(n *Node, init *Nodes) *Node {
                // Zero- or single-element array, of any type.
                switch t.Bound {
                case 0:
-                       n = finishcompare(n, n, Nodbool(n.Op == OEQ), init)
+                       n = finishcompare(n, Nodbool(n.Op == OEQ), init)
                        return n
                case 1:
                        l0 := Nod(OINDEX, l, Nodintconst(0))
                        r0 := Nod(OINDEX, r, Nodintconst(0))
                        a := Nod(n.Op, l0, r0)
-                       n = finishcompare(n, n, a, init)
+                       n = finishcompare(n, a, init)
                        return n
                }
        }
@@ -3242,7 +3242,7 @@ func walkcompare(n *Node, init *Nodes) *Node {
                if expr == nil {
                        expr = Nodbool(n.Op == OEQ)
                }
-               n = finishcompare(n, n, expr, init)
+               n = finishcompare(n, expr, init)
                return n
        }
 
@@ -3260,15 +3260,15 @@ func walkcompare(n *Node, init *Nodes) *Node {
                r = Nod(ONOT, r, nil)
        }
 
-       n = finishcompare(n, n, r, init)
+       n = finishcompare(n, r, init)
        return n
 }
 
-// The result of finishcompare MUST be assigned back to nn, e.g.
+// The result of finishcompare MUST be assigned back to n, e.g.
 //     n.Left = finishcompare(n.Left, x, r, init)
-func finishcompare(nn *Node, n, r *Node, init *Nodes) *Node {
+func finishcompare(n, r *Node, init *Nodes) *Node {
        // Use nn here to avoid passing r to typecheck.
-       nn = r
+       nn := r
        nn = typecheck(nn, Erv)
        nn = walkexpr(nn, init)
        r = nn