diff options
| author | Tobin Harding <[email protected]> | 2017-10-06 13:33:44 +1100 |
|---|---|---|
| committer | Tony Hutter <[email protected]> | 2017-10-16 10:57:55 -0700 |
| commit | 80cc2f6111d4300d5293f7d95d4b0c77d9df8bd9 (patch) | |
| tree | 2308f5749dc84bb691b6328ad4a8e6436048c294 /cmd | |
| parent | b97948276d86343e3f4a9aa1232af9e02106ceeb (diff) | |
Remove unnecessary equality check
Currently `if` statement includes an assignment (from a function return
value) and a equality check. The parenthesis are in the incorrect place,
currently the code clobbers the function return value because of this.
We can fix this by simplifying the `if` statement.
`if (foo != 0)`
can be more succinctly expressed as
`if (foo)`
Remove the equality check, add parenthesis to correct the statement.
Reviewed-by: Brian Behlendorf <[email protected]>
Reviewed-by: George Melikov <[email protected]>
Reviewed-by: Chris Dunlop <[email protected]>
Signed-off-by: Tobin C. Harding <[email protected]>
Closes #6685
Close #6719
Diffstat (limited to 'cmd')
0 files changed, 0 insertions, 0 deletions