qbit/go
1
0
Fork 0
mirror of https://github.com/golang/go synced 2024-11-22 04:34:39 -07:00
Code Releases Activity

cmd/gc: do not corrupt init() with initializers of _ in closures.

Browse Source 
Fixes #5607.

R=golang-dev, daniel.morsing, r, dsymonds
CC=golang-dev
https://golang.org/cl/9952043
This commit is contained in:
Rémy Oudompheng 2013-06-02 23:54:34 +02:00
parent 85046d3e52
commit 0d0d57ccfe
2 changed files with 43 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/cmd/gc/sinit.c
View File

@ -50,8 +50,11 @@ init1(Node *n, NodeList **out)
case PFUNC: case PFUNC:
break; break;
default: default:
if(isblank(n) && n->defn != N && n->defn->initorder == InitNotStarted) { if(isblank(n) && n->curfn == N && n->defn != N && n->defn->initorder == InitNotStarted) {
// blank names initialization is part of init() but not
// when they are inside a function.
n->defn->initorder = InitDone; n->defn->initorder = InitDone;
if(debug['%']) dump("nonstatic", n->defn);
*out = list(*out, n->defn); *out = list(*out, n->defn);
} }
return; return;
@ -62,7 +65,7 @@ init1(Node *n, NodeList **out)
if(n->initorder == InitPending) { if(n->initorder == InitPending) {
if(n->class == PFUNC) if(n->class == PFUNC)
return; return;
// if there have already been errors printed, // if there have already been errors printed,
// those errors probably confused us and // those errors probably confused us and
// there might not be a loop. let the user // there might not be a loop. let the user
@ -128,7 +131,7 @@ init1(Node *n, NodeList **out)
if(debug['j']) if(debug['j'])
print("%S\n", n->sym); print("%S\n", n->sym);
if(!staticinit(n, out)) { if(!staticinit(n, out)) {
if(debug['%']) dump("nonstatic", n->defn); if(debug['%']) dump("nonstatic", n->defn);
*out = list(*out, n->defn); *out = list(*out, n->defn);
} }
} else if(0) { } else if(0) {
@ -149,6 +152,7 @@ if(debug['%']) dump("nonstatic", n->defn);
n->defn->initorder = InitDone; n->defn->initorder = InitDone;
for(l=n->defn->rlist; l; l=l->next) for(l=n->defn->rlist; l; l=l->next)
init1(l->n, out); init1(l->n, out);
if(debug['%']) dump("nonstatic", n->defn);
*out = list(*out, n->defn); *out = list(*out, n->defn);
break; break;
} }

36
test/fixedbugs/issue5607.go Normal file
View File

@ -0,0 +1,36 @@
// run
// Copyright 2013 The Go Authors. All rights reserved.
// Use of this source code is governed by a BSD-style
// license that can be found in the LICENSE file.
// Issue 5607: generation of init() function incorrectly
// uses initializers of blank variables inside closures.
package main
var Test = func() {
var mymap = map[string]string{"a": "b"}
var innerTest = func() {
// Used to crash trying to compile this line as
// part of init() (funcdepth mismatch).
var _, x = mymap["a"]
println(x)
}
innerTest()
}
var Test2 = func() {
// The following initializer should not be part of init()
// The compiler used to generate a call to Panic() in init().
var _, x = Panic()
_ = x
}
func Panic() (int, int) {
panic("omg")
return 1, 2
}
func main() {}