Merge pull request #1037 from Quasilyte/patch-1

mcuadros · web-flow · commit 791aea319719 · 2018-12-12T11:06:24.000+01:00
Simplify return statement in receivePackNoCheck
diff --git a/plumbing/transport/test/receive_pack.go b/plumbing/transport/test/receive_pack.go
@@ -262,11 +262,7 @@ func (s *ReceivePackSuite) receivePackNoCheck(c *C, ep *transport.Endpoint,
 		req.Packfile = s.emptyPackfile()
 	}
 
-	if s, err := r.ReceivePack(context.Background(), req); err != nil {
-		return s, err
-	} else {
-		return s, err
-	}
+	return r.ReceivePack(context.Background(), req)
 }
 
 func (s *ReceivePackSuite) receivePack(c *C, ep *transport.Endpoint,

Original file line number	Diff line number	Diff line change
`@@ -262,11 +262,7 @@ func (s ReceivePackSuite) receivePackNoCheck(c C, ep *transport.Endpoint,`
`262`	`262`	`req.Packfile = s.emptyPackfile()`
`263`	`263`	`}`
`264`	`264`
`265`		`- if s, err := r.ReceivePack(context.Background(), req); err != nil {`
`266`		`- return s, err`
`267`		`- } else {`
`268`		`- return s, err`
`269`		`- }`
	`265`	`+ return r.ReceivePack(context.Background(), req)`
`270`	`266`	`}`
`271`	`267`
`272`	`268`	`func (s ReceivePackSuite) receivePack(c C, ep *transport.Endpoint,`