Skip to content

Commit ccae744

Browse files
committed
cmd/compile/internal/gc: better loop var names in esc.go
Used gorename. Change-Id: Ib33305dc95876ec18e2473ad2999788a32eb21c0 Reviewed-on: https://go-review.googlesource.com/38146 Run-TryBot: Matthew Dempsky <[email protected]> TryBot-Result: Gobot Gobot <[email protected]> Reviewed-by: Brad Fitzpatrick <[email protected]> Reviewed-by: David Chase <[email protected]>
1 parent 7a9aa06 commit ccae744

File tree

1 file changed

+27
-29
lines changed
  • src/cmd/compile/internal/gc

1 file changed

+27
-29
lines changed

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

Lines changed: 27 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -663,10 +663,10 @@ func (e *EscState) esc(n *Node, parent *Node) {
663663
// must happen before processing of switch body,
664664
// so before recursion.
665665
if n.Op == OSWITCH && n.Left != nil && n.Left.Op == OTYPESW {
666-
for _, n1 := range n.List.Slice() { // cases
666+
for _, cas := range n.List.Slice() { // cases
667667
// it.N().Rlist is the variable per case
668-
if n1.Rlist.Len() != 0 {
669-
e.nodeEscState(n1.Rlist.First()).Loopdepth = e.loopdepth
668+
if cas.Rlist.Len() != 0 {
669+
e.nodeEscState(cas.Rlist.First()).Loopdepth = e.loopdepth
670670
}
671671
}
672672
}
@@ -750,12 +750,12 @@ func (e *EscState) esc(n *Node, parent *Node) {
750750

751751
case OSWITCH:
752752
if n.Left != nil && n.Left.Op == OTYPESW {
753-
for _, n2 := range n.List.Slice() {
753+
for _, cas := range n.List.Slice() {
754754
// cases
755755
// n.Left.Right is the argument of the .(type),
756756
// it.N().Rlist is the variable per case
757-
if n2.Rlist.Len() != 0 {
758-
e.escassignWhyWhere(n2.Rlist.First(), n.Left.Right, "switch case", n)
757+
if cas.Rlist.Len() != 0 {
758+
e.escassignWhyWhere(cas.Rlist.First(), n.Left.Right, "switch case", n)
759759
}
760760
}
761761
}
@@ -821,19 +821,17 @@ func (e *EscState) esc(n *Node, parent *Node) {
821821
// TODO: leak to a dummy node instead
822822
// defer f(x) - f and x escape
823823
e.escassignSinkWhy(n, n.Left.Left, "defer func")
824-
825824
e.escassignSinkWhy(n, n.Left.Right, "defer func ...") // ODDDARG for call
826-
for _, n4 := range n.Left.List.Slice() {
827-
e.escassignSinkWhy(n, n4, "defer func arg")
825+
for _, arg := range n.Left.List.Slice() {
826+
e.escassignSinkWhy(n, arg, "defer func arg")
828827
}
829828

830829
case OPROC:
831830
// go f(x) - f and x escape
832831
e.escassignSinkWhy(n, n.Left.Left, "go func")
833-
834832
e.escassignSinkWhy(n, n.Left.Right, "go func ...") // ODDDARG for call
835-
for _, n4 := range n.Left.List.Slice() {
836-
e.escassignSinkWhy(n, n4, "go func arg")
833+
for _, arg := range n.Left.List.Slice() {
834+
e.escassignSinkWhy(n, arg, "go func arg")
837835
}
838836

839837
case OCALLMETH, OCALLFUNC, OCALLINTER:
@@ -908,28 +906,28 @@ func (e *EscState) esc(n *Node, parent *Node) {
908906

909907
case OARRAYLIT:
910908
// Link values to array
911-
for _, n2 := range n.List.Slice() {
912-
if n2.Op == OKEY {
913-
n2 = n2.Right
909+
for _, elt := range n.List.Slice() {
910+
if elt.Op == OKEY {
911+
elt = elt.Right
914912
}
915-
e.escassign(n, n2, e.stepAssignWhere(n, n2, "array literal element", n))
913+
e.escassign(n, elt, e.stepAssignWhere(n, elt, "array literal element", n))
916914
}
917915

918916
case OSLICELIT:
919917
// Slice is not leaked until proven otherwise
920918
e.track(n)
921919
// Link values to slice
922-
for _, n2 := range n.List.Slice() {
923-
if n2.Op == OKEY {
924-
n2 = n2.Right
920+
for _, elt := range n.List.Slice() {
921+
if elt.Op == OKEY {
922+
elt = elt.Right
925923
}
926-
e.escassign(n, n2, e.stepAssignWhere(n, n2, "slice literal element", n))
924+
e.escassign(n, elt, e.stepAssignWhere(n, elt, "slice literal element", n))
927925
}
928926

929927
// Link values to struct.
930928
case OSTRUCTLIT:
931-
for _, n6 := range n.List.Slice() {
932-
e.escassignWhyWhere(n, n6.Left, "struct literal element", n)
929+
for _, elt := range n.List.Slice() {
930+
e.escassignWhyWhere(n, elt.Left, "struct literal element", n)
933931
}
934932

935933
case OPTRLIT:
@@ -947,9 +945,9 @@ func (e *EscState) esc(n *Node, parent *Node) {
947945
case OMAPLIT:
948946
e.track(n)
949947
// Keys and values make it to memory, lose track.
950-
for _, n7 := range n.List.Slice() {
951-
e.escassignSinkWhy(n, n7.Left, "map literal key")
952-
e.escassignSinkWhy(n, n7.Right, "map literal value")
948+
for _, elt := range n.List.Slice() {
949+
e.escassignSinkWhy(n, elt.Left, "map literal key")
950+
e.escassignSinkWhy(n, elt.Right, "map literal value")
953951
}
954952

955953
case OCLOSURE:
@@ -1942,11 +1940,11 @@ func (e *EscState) escwalkBody(level Level, dst *Node, src *Node, step *EscStep,
19421940
level = level.dec()
19431941

19441942
case OSLICELIT:
1945-
for _, n1 := range src.List.Slice() {
1946-
if n1.Op == OKEY {
1947-
n1 = n1.Right
1943+
for _, elt := range src.List.Slice() {
1944+
if elt.Op == OKEY {
1945+
elt = elt.Right
19481946
}
1949-
e.escwalk(level.dec(), dst, n1, e.stepWalk(dst, n1, "slice-literal-element", step))
1947+
e.escwalk(level.dec(), dst, elt, e.stepWalk(dst, elt, "slice-literal-element", step))
19501948
}
19511949

19521950
fallthrough

0 commit comments

Comments
 (0)