From 6a228239392fd4f3a3d04c816be4fdd39d3a36b1 Mon Sep 17 00:00:00 2001 From: Robert Griesemer Date: Mon, 30 Jun 2014 14:40:12 -0700 Subject: [PATCH] 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 --- src/cmd/gofmt/rewrite.go | 3 --- 1 file changed, 3 deletions(-) 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 -- 2.48.1