Suspicious error handling in buildFromType()

[seiyab]

What does this post aims
I possibly found a tiny error so report it. I want to know whether it is intentional or not out of curiosity.

What I found
I'm reading codes in this repo and found a suspicious error handling in buildFromType()

go-swagger/codescan/schema.go

Lines 291 to 302 in 1182d39

	func (s *schemaBuilder) buildFromType(tpe types.Type, tgt swaggerTypable) error {
	pkg, err := importer.Default().Import("encoding")
	if err != nil {
	return nil
	}
	ifc := pkg.Scope().Lookup("TextMarshaler").Type().Underlying().(*types.Interface)
	// check if the type implements encoding.TextMarshaler interface
	isTextMarshaler := types.Implements(tpe, ifc)
	if isTextMarshaler {
	return s.buildFromTextMarshal(tpe, tgt)
	}

I think if err != nil { return nil } looks unintentional and possibly a (unreproducible?) bug. I suggest something like

pkg, err := importer.Default().Import("encoding") 
if err == nil { 
 	ifc := pkg.Scope().Lookup("TextMarshaler").Type().Underlying().(*types.Interface) 

 	// check if the type implements encoding.TextMarshaler interface 
 	isTextMarshaler := types.Implements(tpe, ifc) 
 	if isTextMarshaler { 
 	 	return s.buildFromTextMarshal(tpe, tgt) 
 	}
} 

looks a natural implementation.