Remove unreachable and duplicated return statement

puellanivis · web-flow · commit a17a626ab843 · 2022-03-30T09:17:11.000Z
I’m confident blame will show I am at fault here, not sure how I missed it, and why `go vet` didn’t catch it either.
diff --git a/internal/encoding/ssh/filexfer/attrs.go b/internal/encoding/ssh/filexfer/attrs.go
@@ -74,7 +74,6 @@ func (a *Attributes) SetPermissions(perms FileMode) {
 // GetACModTime returns the ATime and MTime fields and a bool that is true if and only if the values are valid/defined.
 func (a *Attributes) GetACModTime() (atime, mtime uint32, ok bool) {
 	return a.ATime, a.MTime, a.Flags&AttrACModTime != 0
-	return a.ATime, a.MTime, a.Flags&AttrACModTime != 0
 }
 
 // SetACModTime is a convenience function that sets the ATime and MTime fields,

Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,6 @@ func (a *Attributes) SetPermissions(perms FileMode) {`
`74`	`74`	`// GetACModTime returns the ATime and MTime fields and a bool that is true if and only if the values are valid/defined.`
`75`	`75`	`func (a *Attributes) GetACModTime() (atime, mtime uint32, ok bool) {`
`76`	`76`	`return a.ATime, a.MTime, a.Flags&AttrACModTime != 0`
`77`		`- return a.ATime, a.MTime, a.Flags&AttrACModTime != 0`
`78`	`77`	`}`
`79`	`78`
`80`	`79`	`// SetACModTime is a convenience function that sets the ATime and MTime fields,`