Redundant error checking RVV-B0005
Style
Major
2 years ago
—
2 years old
redundant if ...; err != nil check, just return error instead.
49 }
50
51 if ok, _ := client.Extension("AUTH"); ok {
52 if err = client.Auth(auth); err != nil {53 return err
54 }
55 return nil
Description
Some simple error checking patterns can be reduced to just a single return
statement returning a variable of type Error.
Bad practice
if err != nil {
return err
}
return nil
Recommended
return err