From: Robert Griesemer Date: Mon, 30 Jun 2014 21:40:12 +0000 (-0700) Subject: gofmt: remove redundant check in rewriter X-Git-Tag: go1.4beta1~1196 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=6a228239392fd4f3a3d04c816be4fdd39d3a36b1;p=gostls13.git gofmt: remove redundant check in rewriter If the actual types of two reflect values are the same and the values are structs, they must have the same number of fields. LGTM=rsc R=rsc CC=golang-codereviews https://golang.org/cl/108280043 --- diff --git a/src/cmd/gofmt/rewrite.go b/src/cmd/gofmt/rewrite.go index fb6c6fc811..d267cfcc1d 100644 --- a/src/cmd/gofmt/rewrite.go +++ b/src/cmd/gofmt/rewrite.go @@ -226,9 +226,6 @@ func match(m map[string]reflect.Value, pattern, val reflect.Value) bool { return true case reflect.Struct: - if p.NumField() != v.NumField() { - return false - } for i := 0; i < p.NumField(); i++ { if !match(m, p.Field(i), v.Field(i)) { return false