From 90ea57ac7c751041f2eef5354c80cb997b33588d Mon Sep 17 00:00:00 2001 From: "deepsource-autofix-local[bot]" <61578317+deepsource-autofix-local[bot]@users.noreply.github.com> Date: Tue, 6 Apr 2021 16:53:24 +0000 Subject: [PATCH] Omit redundant `nil` check in type assertion --- pkg/proc/proc_test.go | 7 +++---- pkg/proc/target_exec.go | 7 +++---- 2 files changed, 6 insertions(+), 8 deletions(-) diff --git a/pkg/proc/proc_test.go b/pkg/proc/proc_test.go index c25f779932..33887a84ed 100644 --- a/pkg/proc/proc_test.go +++ b/pkg/proc/proc_test.go @@ -1848,11 +1848,10 @@ func TestIssue414(t *testing.T) { } else { err = p.Next() } - if err != nil { - if _, exited := err.(proc.ErrProcessExited); exited { - break - } + if _, exited := err.(proc.ErrProcessExited); exited { + break } + assertNoError(err, t, "Step()") } }) diff --git a/pkg/proc/target_exec.go b/pkg/proc/target_exec.go index 385ce9e25c..9dd71abe74 100644 --- a/pkg/proc/target_exec.go +++ b/pkg/proc/target_exec.go @@ -782,11 +782,10 @@ func setStepIntoBreakpoint(dbp *Target, curfn *Function, text []AsmInstruction, } func allowDuplicateBreakpoint(bp *Breakpoint, err error) (*Breakpoint, error) { - if err != nil { - if _, isexists := err.(BreakpointExistsError); isexists { - return bp, nil - } + if _, isexists := err.(BreakpointExistsError); isexists { + return bp, nil } + return bp, err }