]> Cypherpunks repositories - gostls13.git/commit
[dev.typeparams] cmd/compile/internal/types2: review of initorder.go
authorRobert Griesemer <gri@golang.org>
Tue, 27 Oct 2020 18:26:54 +0000 (11:26 -0700)
committerRobert Griesemer <gri@golang.org>
Tue, 27 Oct 2020 23:30:32 +0000 (23:30 +0000)
commit6e98406ac3e654f4df15f662f51eda46434af332
tree2ced7c94e1b246e364239e35300ee6abc136edbe
parent38af45b4cb4ce42b338ba9960419684b2c2c5e72
[dev.typeparams] cmd/compile/internal/types2: review of initorder.go

Difference: errorf now accepts any value that implements
the poser interface in place of a position argument. All
types2 Objects implement poser.

type poser interface {
Pos() syntax.Pos
}

f=initorder.go; diff $f ../../../../go/types/$f
5c5
< package types2
---
> package types
154c154
<  check.errorf(obj, "initialization cycle for %s", obj.Name())
---
>  check.errorf(obj.Pos(), "initialization cycle for %s", obj.Name())
157c157
<  check.errorf(obj, "\t%s refers to", obj.Name()) // secondary error, \t indented
---
>  check.errorf(obj.Pos(), "\t%s refers to", obj.Name()) // secondary error, \t indented
161c161
<  check.errorf(obj, "\t%s", obj.Name())
---
>  check.errorf(obj.Pos(), "\t%s", obj.Name())

Change-Id: Id85074fd15a04bb4ff6e8b68a44be6ac5919c71a
Reviewed-on: https://go-review.googlesource.com/c/go/+/265678
Trust: Robert Griesemer <gri@golang.org>
Reviewed-by: Robert Findley <rfindley@google.com>
src/cmd/compile/internal/types2/initorder.go