]> Cypherpunks repositories - gostls13.git/commitdiff
gc: export nil literals without inferred type.
authorLuuk van Dijk <lvd@golang.org>
Wed, 11 Jan 2012 20:26:54 +0000 (21:26 +0100)
committerLuuk van Dijk <lvd@golang.org>
Wed, 11 Jan 2012 20:26:54 +0000 (21:26 +0100)
Fixes #2678

R=rsc
CC=golang-dev
https://golang.org/cl/5529066

src/cmd/gc/fmt.c
test/fixedbugs/bug392.dir/one.go [new file with mode: 0644]
test/fixedbugs/bug392.dir/two.go [new file with mode: 0644]
test/fixedbugs/bug392.go [new file with mode: 0644]

index 10bf02130a480f1ef0e4d905a190603efbecae4c..0eb5854aec55e065c33f96575f7955c63a0112b4 100644 (file)
@@ -1072,9 +1072,11 @@ exprfmt(Fmt *f, Node *n, int prec)
        case OREGISTER:
                return fmtprint(f, "%R", n->val.u.reg);
 
-       case OLITERAL:  // this is still a bit of a mess
+       case OLITERAL:  // this is a bit of a mess
                if(fmtmode == FErr && n->sym != S)
                        return fmtprint(f, "%S", n->sym);
+               if(n->val.ctype == CTNIL)
+                       n = n->orig; // if this node was a nil decorated with at type, print the original naked nil
                if(n->type != types[n->type->etype] && n->type != idealbool && n->type != idealstring) {
                        if(isptr[n->type->etype])
                                return fmtprint(f, "(%T)(%V)", n->type, &n->val);
diff --git a/test/fixedbugs/bug392.dir/one.go b/test/fixedbugs/bug392.dir/one.go
new file mode 100644 (file)
index 0000000..6353b88
--- /dev/null
@@ -0,0 +1,6 @@
+package one
+
+type file int
+func (file *file) isnil() bool { return file == nil }
+func (fil *file)  isnil2() bool { return fil == nil }
+
diff --git a/test/fixedbugs/bug392.dir/two.go b/test/fixedbugs/bug392.dir/two.go
new file mode 100644 (file)
index 0000000..8a5346a
--- /dev/null
@@ -0,0 +1,4 @@
+package two
+
+import _ "./one"
+
diff --git a/test/fixedbugs/bug392.go b/test/fixedbugs/bug392.go
new file mode 100644 (file)
index 0000000..46781eb
--- /dev/null
@@ -0,0 +1,11 @@
+// $G $D/$F.dir/one.go && $G -ll $D/$F.dir/two.go
+
+// Copyright 2011 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 2678
+// -ll flag in command above is to force typecheck on import, needed to trigger the bug.
+// fixedbugs/bug392.dir/two.go:3: cannot call non-function *one.file (type one.file)
+
+package ignored