From: Rob Pike <r@golang.org>
Date: Wed, 9 Sep 2015 18:01:01 +0000 (-0700)
Subject: cmd/doc: the builtin package should always show unexported symbols
X-Git-Tag: go1.6beta1~1130
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=cf3134a0178c1c5a2c1b523dbdfa953dbf60481b;p=gostls13.git

cmd/doc: the builtin package should always show unexported symbols

Trivial fix: set unexported=true for builtin.
Godoc itself has a similar hack.

Fixes #12541

Change-Id: Ib701f867d117931eb6ec6de223941b52eb6cd4a7
Reviewed-on: https://go-review.googlesource.com/14441
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
---

diff --git a/src/cmd/doc/main.go b/src/cmd/doc/main.go
index 0985d95b60..1f503e92fa 100644
--- a/src/cmd/doc/main.go
+++ b/src/cmd/doc/main.go
@@ -87,6 +87,7 @@ func do(writer io.Writer, flagSet *flag.FlagSet, args []string) (err error) {
 	buildPackage, userPath, symbol := parseArgs(flagSet.Args())
 	symbol, method := parseSymbol(symbol)
 	pkg := parsePackage(writer, buildPackage, userPath)
+
 	defer func() {
 		pkg.flush()
 		e := recover()
@@ -100,6 +101,13 @@ func do(writer io.Writer, flagSet *flag.FlagSet, args []string) (err error) {
 		}
 		panic(e)
 	}()
+
+	// The builtin package needs special treatment: its symbols are lower
+	// case but we want to see them, always.
+	if pkg.build.ImportPath == "builtin" {
+		unexported = true
+	}
+
 	switch {
 	case symbol == "":
 		pkg.packageDoc()