diff options
| author | Tobin Harding <[email protected]> | 2017-10-06 13:33:44 +1100 |
|---|---|---|
| committer | Brian Behlendorf <[email protected]> | 2017-10-05 19:33:44 -0700 |
| commit | d95a59805f8f00046bb449fb12161e1f0caf65fb (patch) | |
| tree | d9a41c964237a86d9c360e332e7067104456cf14 /cmd/zed | |
| parent | eea2e2413220b34a3da0911dee554df20bdd0c47 (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/zed')
0 files changed, 0 insertions, 0 deletions