From 8bdc76f8b89a95f0516b0578abcdb04476e759db Mon Sep 17 00:00:00 2001 From: Greg Mankes Date: Mon, 28 Oct 2019 17:48:11 -0400 Subject: [PATCH] remove redundant code no need to handle error condition from client response, just return the response and error directly --- databases.go | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/databases.go b/databases.go index 206841e..ccffff8 100644 --- a/databases.go +++ b/databases.go @@ -699,11 +699,7 @@ func (svc *DatabasesServiceOp) GetFirewallRules(ctx context.Context, databaseID if err != nil { return nil, err } - resp, err := svc.client.Do(ctx, req, root) - if err != nil { - return resp, err - } - return resp, nil + return svc.client.Do(ctx, req, root) } // UpdateFirewallRules sets the inbound sources for a given cluster. @@ -713,9 +709,5 @@ func (svc *DatabasesServiceOp) UpdateFirewallRules(ctx context.Context, database if err != nil { return nil, err } - resp, err := svc.client.Do(ctx, req, nil) - if err != nil { - return resp, err - } - return resp, nil + return svc.client.Do(ctx, req, nil) }