diff options
| author | Rusty Russell <rusty@rustcorp.com.au> | 2011-12-19 13:56:45 (GMT) |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2011-12-20 03:27:29 (GMT) |
| commit | 3db1cd5c05f35fb43eb134df6f321de4e63141f2 (patch) | |
| tree | 960039f3f4f0a524b37e94434624da154859bc64 /drivers/net/wireless/b43/xmit.c | |
| parent | a8e510f682fe6d7671c11887e07c55f86caaf3c1 (diff) | |
| download | linux-fsl-qoriq-3db1cd5c05f35fb43eb134df6f321de4e63141f2.tar.xz | |
net: fix assignment of 0/1 to bool variables.
DaveM said:
Please, this kind of stuff rots forever and not using bool properly
drives me crazy.
Joe Perches <joe@perches.com> gave me the spatch script:
@@
bool b;
@@
-b = 0
+b = false
@@
bool b;
@@
-b = 1
+b = true
I merely installed coccinelle, read the documentation and took credit.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/wireless/b43/xmit.c')
| -rw-r--r-- | drivers/net/wireless/b43/xmit.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/wireless/b43/xmit.c b/drivers/net/wireless/b43/xmit.c index 5f77cbe..2c53678 100644 --- a/drivers/net/wireless/b43/xmit.c +++ b/drivers/net/wireless/b43/xmit.c @@ -874,7 +874,7 @@ bool b43_fill_txstatus_report(struct b43_wldev *dev, struct ieee80211_tx_info *report, const struct b43_txstatus *status) { - bool frame_success = 1; + bool frame_success = true; int retry_limit; /* preserve the confiured retry limit before clearing the status @@ -890,7 +890,7 @@ bool b43_fill_txstatus_report(struct b43_wldev *dev, /* The frame was not ACKed... */ if (!(report->flags & IEEE80211_TX_CTL_NO_ACK)) { /* ...but we expected an ACK. */ - frame_success = 0; + frame_success = false; } } if (status->frame_count == 0) { |