Redundant error checking RVV-B0005
Style
Major
8 months ago
—
2 years old
redundant if ...; err != nil check, just return error instead.
43 return c.conn.WriteJSON(msg)
44}
45func (msg *Message) UnmarshalBinary(data []byte) error {
46 if err := json.Unmarshal(data, &msg); err != nil { 47 return err
48 }
49 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