]> Cypherpunks.ru repositories - gostls13.git/blobdiff - src/cmd/compile/internal/types2/errors.go
go/types, types2: remove local version processing in favor of go/version
[gostls13.git] / src / cmd / compile / internal / types2 / errors.go
index 941e7c6fd36a5ca5331fc7a0ae12f73c16f7f9a2..b8414b48498f242b088f5f8d1517156de31db1e4 100644 (file)
@@ -1,4 +1,3 @@
-// UNREVIEWED
 // Copyright 2012 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.
@@ -11,17 +10,21 @@ import (
        "bytes"
        "cmd/compile/internal/syntax"
        "fmt"
+       . "internal/types/errors"
+       "runtime"
        "strconv"
        "strings"
 )
 
-func unimplemented() {
-       panic("unimplemented")
-}
-
 func assert(p bool) {
        if !p {
-               panic("assertion failed")
+               msg := "assertion failed"
+               // Include information about the assertion location. Due to panic recovery,
+               // this location is otherwise buried in the middle of the panicking stack.
+               if _, file, line, ok := runtime.Caller(1); ok {
+                       msg = fmt.Sprintf("%s:%d: %s", file, line, msg)
+               }
+               panic(msg)
        }
 }
 
@@ -29,55 +32,202 @@ func unreachable() {
        panic("unreachable")
 }
 
-func (check *Checker) qualifier(pkg *Package) string {
-       // Qualify the package unless it's the package being type-checked.
-       if pkg != check.pkg {
-               // If the same package name was used by multiple packages, display the full path.
-               if check.pkgCnt[pkg.name] > 1 {
-                       return strconv.Quote(pkg.path)
+// An error_ represents a type-checking error.
+// To report an error_, call Checker.report.
+type error_ struct {
+       desc []errorDesc
+       code Code
+       soft bool // TODO(gri) eventually determine this from an error code
+}
+
+// An errorDesc describes part of a type-checking error.
+type errorDesc struct {
+       pos    syntax.Pos
+       format string
+       args   []interface{}
+}
+
+func (err *error_) empty() bool {
+       return err.desc == nil
+}
+
+func (err *error_) pos() syntax.Pos {
+       if err.empty() {
+               return nopos
+       }
+       return err.desc[0].pos
+}
+
+func (err *error_) msg(qf Qualifier) string {
+       if err.empty() {
+               return "no error"
+       }
+       var buf strings.Builder
+       for i := range err.desc {
+               p := &err.desc[i]
+               if i > 0 {
+                       fmt.Fprint(&buf, "\n\t")
+                       if p.pos.IsKnown() {
+                               fmt.Fprintf(&buf, "%s: ", p.pos)
+                       }
                }
-               return pkg.name
+               buf.WriteString(sprintf(qf, false, p.format, p.args...))
        }
-       return ""
+       return buf.String()
 }
 
-func (check *Checker) sprintf(format string, args ...interface{}) string {
+// String is for testing.
+func (err *error_) String() string {
+       if err.empty() {
+               return "no error"
+       }
+       return fmt.Sprintf("%s: %s", err.pos(), err.msg(nil))
+}
+
+// errorf adds formatted error information to err.
+// It may be called multiple times to provide additional information.
+func (err *error_) errorf(at poser, format string, args ...interface{}) {
+       err.desc = append(err.desc, errorDesc{atPos(at), format, args})
+}
+
+func sprintf(qf Qualifier, tpSubscripts bool, format string, args ...interface{}) string {
        for i, arg := range args {
                switch a := arg.(type) {
                case nil:
                        arg = "<nil>"
                case operand:
-                       panic("internal error: should always pass *operand")
+                       panic("got operand instead of *operand")
                case *operand:
-                       arg = operandString(a, check.qualifier)
+                       arg = operandString(a, qf)
                case syntax.Pos:
                        arg = a.String()
                case syntax.Expr:
-                       arg = ExprString(a)
+                       arg = syntax.String(a)
+               case []syntax.Expr:
+                       var buf strings.Builder
+                       buf.WriteByte('[')
+                       for i, x := range a {
+                               if i > 0 {
+                                       buf.WriteString(", ")
+                               }
+                               buf.WriteString(syntax.String(x))
+                       }
+                       buf.WriteByte(']')
+                       arg = buf.String()
                case Object:
-                       arg = ObjectString(a, check.qualifier)
+                       arg = ObjectString(a, qf)
                case Type:
-                       arg = TypeString(a, check.qualifier)
+                       var buf bytes.Buffer
+                       w := newTypeWriter(&buf, qf)
+                       w.tpSubscripts = tpSubscripts
+                       w.typ(a)
+                       arg = buf.String()
+               case []Type:
+                       var buf bytes.Buffer
+                       w := newTypeWriter(&buf, qf)
+                       w.tpSubscripts = tpSubscripts
+                       buf.WriteByte('[')
+                       for i, x := range a {
+                               if i > 0 {
+                                       buf.WriteString(", ")
+                               }
+                               w.typ(x)
+                       }
+                       buf.WriteByte(']')
+                       arg = buf.String()
+               case []*TypeParam:
+                       var buf bytes.Buffer
+                       w := newTypeWriter(&buf, qf)
+                       w.tpSubscripts = tpSubscripts
+                       buf.WriteByte('[')
+                       for i, x := range a {
+                               if i > 0 {
+                                       buf.WriteString(", ")
+                               }
+                               w.typ(x)
+                       }
+                       buf.WriteByte(']')
+                       arg = buf.String()
                }
                args[i] = arg
        }
        return fmt.Sprintf(format, args...)
 }
 
+func (check *Checker) qualifier(pkg *Package) string {
+       // Qualify the package unless it's the package being type-checked.
+       if pkg != check.pkg {
+               if check.pkgPathMap == nil {
+                       check.pkgPathMap = make(map[string]map[string]bool)
+                       check.seenPkgMap = make(map[*Package]bool)
+                       check.markImports(check.pkg)
+               }
+               // If the same package name was used by multiple packages, display the full path.
+               if len(check.pkgPathMap[pkg.name]) > 1 {
+                       return strconv.Quote(pkg.path)
+               }
+               return pkg.name
+       }
+       return ""
+}
+
+// markImports recursively walks pkg and its imports, to record unique import
+// paths in pkgPathMap.
+func (check *Checker) markImports(pkg *Package) {
+       if check.seenPkgMap[pkg] {
+               return
+       }
+       check.seenPkgMap[pkg] = true
+
+       forName, ok := check.pkgPathMap[pkg.name]
+       if !ok {
+               forName = make(map[string]bool)
+               check.pkgPathMap[pkg.name] = forName
+       }
+       forName[pkg.path] = true
+
+       for _, imp := range pkg.imports {
+               check.markImports(imp)
+       }
+}
+
+// check may be nil.
+func (check *Checker) sprintf(format string, args ...interface{}) string {
+       var qf Qualifier
+       if check != nil {
+               qf = check.qualifier
+       }
+       return sprintf(qf, false, format, args...)
+}
+
+func (check *Checker) report(err *error_) {
+       if err.empty() {
+               panic("no error to report")
+       }
+       check.err(err.pos(), err.code, err.msg(check.qualifier), err.soft)
+}
+
 func (check *Checker) trace(pos syntax.Pos, format string, args ...interface{}) {
        fmt.Printf("%s:\t%s%s\n",
                pos,
                strings.Repeat(".  ", check.indent),
-               check.sprintf(format, args...),
+               sprintf(check.qualifier, true, format, args...),
        )
 }
 
 // dump is only needed for debugging
 func (check *Checker) dump(format string, args ...interface{}) {
-       fmt.Println(check.sprintf(format, args...))
+       fmt.Println(sprintf(check.qualifier, true, format, args...))
 }
 
-func (check *Checker) err(pos syntax.Pos, msg string, soft bool) {
+func (check *Checker) err(at poser, code Code, msg string, soft bool) {
+       switch code {
+       case InvalidSyntaxTree:
+               msg = "invalid syntax tree: " + msg
+       case 0:
+               panic("no error code provided")
+       }
+
        // Cheap trick: Don't report errors with messages containing
        // "invalid operand" or "invalid type" as those tend to be
        // follow-on errors which don't add useful information. Only
@@ -87,6 +237,8 @@ func (check *Checker) err(pos syntax.Pos, msg string, soft bool) {
                return
        }
 
+       pos := atPos(at)
+
        // If we are encountering an error while evaluating an inherited
        // constant initialization expression, pos is the position of in
        // the original expression, and not of the currently declared
@@ -98,7 +250,17 @@ func (check *Checker) err(pos syntax.Pos, msg string, soft bool) {
                pos = check.errpos
        }
 
-       err := Error{pos, stripAnnotations(msg), msg, soft}
+       // If we have a URL for error codes, add a link to the first line.
+       if code != 0 && check.conf.ErrorURL != "" {
+               u := fmt.Sprintf(check.conf.ErrorURL, code)
+               if i := strings.Index(msg, "\n"); i >= 0 {
+                       msg = msg[:i] + u + msg[i:]
+               } else {
+                       msg += u
+               }
+       }
+
+       err := Error{pos, stripAnnotations(msg), msg, soft, code}
        if check.firstErr == nil {
                check.firstErr = err
        }
@@ -114,59 +276,57 @@ func (check *Checker) err(pos syntax.Pos, msg string, soft bool) {
        f(err)
 }
 
+const (
+       invalidArg = "invalid argument: "
+       invalidOp  = "invalid operation: "
+)
+
 type poser interface {
        Pos() syntax.Pos
 }
 
-func (check *Checker) error(at poser, msg string) {
-       check.err(posFor(at), msg, false)
-}
-
-func (check *Checker) errorf(at poser, format string, args ...interface{}) {
-       check.err(posFor(at), check.sprintf(format, args...), false)
-}
-
-func (check *Checker) softErrorf(at poser, format string, args ...interface{}) {
-       check.err(posFor(at), check.sprintf(format, args...), true)
+func (check *Checker) error(at poser, code Code, msg string) {
+       check.err(at, code, msg, false)
 }
 
-func (check *Checker) invalidASTf(at poser, format string, args ...interface{}) {
-       check.errorf(at, "invalid AST: "+format, args...)
+func (check *Checker) errorf(at poser, code Code, format string, args ...interface{}) {
+       check.err(at, code, check.sprintf(format, args...), false)
 }
 
-func (check *Checker) invalidArgf(at poser, format string, args ...interface{}) {
-       check.errorf(at, "invalid argument: "+format, args...)
+func (check *Checker) softErrorf(at poser, code Code, format string, args ...interface{}) {
+       check.err(at, code, check.sprintf(format, args...), true)
 }
 
-func (check *Checker) invalidOpf(at poser, format string, args ...interface{}) {
-       check.errorf(at, "invalid operation: "+format, args...)
+func (check *Checker) versionErrorf(at poser, v goVersion, format string, args ...interface{}) {
+       msg := check.sprintf(format, args...)
+       msg = fmt.Sprintf("%s requires %s or later", msg, v)
+       check.err(at, UnsupportedFeature, msg, true)
 }
 
-// posFor reports the left (= start) position of at.
-func posFor(at poser) syntax.Pos {
+// atPos reports the left (= start) position of at.
+func atPos(at poser) syntax.Pos {
        switch x := at.(type) {
        case *operand:
                if x.expr != nil {
-                       return startPos(x.expr)
+                       return syntax.StartPos(x.expr)
                }
        case syntax.Node:
-               return startPos(x)
+               return syntax.StartPos(x)
        }
        return at.Pos()
 }
 
 // stripAnnotations removes internal (type) annotations from s.
 func stripAnnotations(s string) string {
-       // Would like to use strings.Builder but it's not available in Go 1.4.
-       var b bytes.Buffer
+       var buf strings.Builder
        for _, r := range s {
                // strip #'s and subscript digits
-               if r != instanceMarker && !('₀' <= r && r < '₀'+10) { // '₀' == U+2080
-                       b.WriteRune(r)
+               if r < '₀' || '₀'+10 <= r { // '₀' == U+2080
+                       buf.WriteRune(r)
                }
        }
-       if b.Len() < len(s) {
-               return b.String()
+       if buf.Len() < len(s) {
+               return buf.String()
        }
        return s
 }