* [PATCH 2/5] Bluetooth: Fix assignment of 0/1 to bool variables
2013-09-22 18:44 [PATCH 1/5] scsi: Fix assignment of 0/1 to bool variables Peter Senna Tschudin
@ 2013-09-22 18:44 ` Peter Senna Tschudin
2013-09-22 18:58 ` Marcel Holtmann
2013-09-22 22:21 ` Gustavo Padovan
2013-09-22 18:44 ` [PATCH 3/5] OMAPDSS: DISPC: " Peter Senna Tschudin
` (3 subsequent siblings)
4 siblings, 2 replies; 10+ messages in thread
From: Peter Senna Tschudin @ 2013-09-22 18:44 UTC (permalink / raw)
To: marcel
Cc: gustavo, johan.hedberg, davem, linux-bluetooth, netdev,
linux-kernel, kernel-janitors, Peter Senna Tschudin
Convert 0 to false and 1 to true when assigning values to bool
variables. Inspired by commit 3db1cd5c05f35fb43eb134df6f321de4e63141f2.
The simplified semantic patch that find this problem is as
follows (http://coccinelle.lip6.fr/):
@@
bool b;
@@
(
-b = 0
+b = false
|
-b = 1
+b = true
)
Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
---
net/bluetooth/l2cap_core.c | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
diff -u -p a/net/bluetooth/l2cap_core.c b/net/bluetooth/l2cap_core.c
--- a/net/bluetooth/l2cap_core.c
+++ b/net/bluetooth/l2cap_core.c
@@ -5797,7 +5797,7 @@ static int l2cap_rx_state_recv(struct l2
struct sk_buff *skb, u8 event)
{
int err = 0;
- bool skb_in_use = 0;
+ bool skb_in_use = false;
BT_DBG("chan %p, control %p, skb %p, event %d", chan, control, skb,
event);
@@ -5818,7 +5818,7 @@ static int l2cap_rx_state_recv(struct l2
control->txseq);
chan->buffer_seq = chan->expected_tx_seq;
- skb_in_use = 1;
+ skb_in_use = true;
err = l2cap_reassemble_sdu(chan, skb, control);
if (err)
@@ -5854,7 +5854,7 @@ static int l2cap_rx_state_recv(struct l2
* current frame is stored for later use.
*/
skb_queue_tail(&chan->srej_q, skb);
- skb_in_use = 1;
+ skb_in_use = true;
BT_DBG("Queued %p (queue len %d)", skb,
skb_queue_len(&chan->srej_q));
@@ -5932,7 +5932,7 @@ static int l2cap_rx_state_srej_sent(stru
{
int err = 0;
u16 txseq = control->txseq;
- bool skb_in_use = 0;
+ bool skb_in_use = false;
BT_DBG("chan %p, control %p, skb %p, event %d", chan, control, skb,
event);
@@ -5944,7 +5944,7 @@ static int l2cap_rx_state_srej_sent(stru
/* Keep frame for reassembly later */
l2cap_pass_to_tx(chan, control);
skb_queue_tail(&chan->srej_q, skb);
- skb_in_use = 1;
+ skb_in_use = true;
BT_DBG("Queued %p (queue len %d)", skb,
skb_queue_len(&chan->srej_q));
@@ -5955,7 +5955,7 @@ static int l2cap_rx_state_srej_sent(stru
l2cap_pass_to_tx(chan, control);
skb_queue_tail(&chan->srej_q, skb);
- skb_in_use = 1;
+ skb_in_use = true;
BT_DBG("Queued %p (queue len %d)", skb,
skb_queue_len(&chan->srej_q));
@@ -5970,7 +5970,7 @@ static int l2cap_rx_state_srej_sent(stru
* the missing frames.
*/
skb_queue_tail(&chan->srej_q, skb);
- skb_in_use = 1;
+ skb_in_use = true;
BT_DBG("Queued %p (queue len %d)", skb,
skb_queue_len(&chan->srej_q));
@@ -5984,7 +5984,7 @@ static int l2cap_rx_state_srej_sent(stru
* SREJ'd frames.
*/
skb_queue_tail(&chan->srej_q, skb);
- skb_in_use = 1;
+ skb_in_use = true;
BT_DBG("Queued %p (queue len %d)", skb,
skb_queue_len(&chan->srej_q));
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH 2/5] Bluetooth: Fix assignment of 0/1 to bool variables
2013-09-22 18:44 ` [PATCH 2/5] Bluetooth: " Peter Senna Tschudin
@ 2013-09-22 18:58 ` Marcel Holtmann
2013-09-22 22:21 ` Gustavo Padovan
1 sibling, 0 replies; 10+ messages in thread
From: Marcel Holtmann @ 2013-09-22 18:58 UTC (permalink / raw)
To: Peter Senna Tschudin
Cc: gustavo, johan.hedberg, davem, linux-bluetooth, netdev,
linux-kernel, kernel-janitors
Hi Peter,
> Convert 0 to false and 1 to true when assigning values to bool
> variables. Inspired by commit 3db1cd5c05f35fb43eb134df6f321de4e63141f2.
>
> The simplified semantic patch that find this problem is as
> follows (http://coccinelle.lip6.fr/):
>
> @@
> bool b;
> @@
> (
> -b = 0
> +b = false
> |
> -b = 1
> +b = true
> )
>
> Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
>
> ---
> net/bluetooth/l2cap_core.c | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
Acked-by: Marcel Holtmann <marcel@holtmann.org>
Regards
Marcel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/5] Bluetooth: Fix assignment of 0/1 to bool variables
2013-09-22 18:44 ` [PATCH 2/5] Bluetooth: " Peter Senna Tschudin
2013-09-22 18:58 ` Marcel Holtmann
@ 2013-09-22 22:21 ` Gustavo Padovan
1 sibling, 0 replies; 10+ messages in thread
From: Gustavo Padovan @ 2013-09-22 22:21 UTC (permalink / raw)
To: Peter Senna Tschudin
Cc: marcel, johan.hedberg, davem, linux-bluetooth, netdev,
linux-kernel, kernel-janitors
Hi Peter,
2013-09-22 Peter Senna Tschudin <peter.senna@gmail.com>:
> Convert 0 to false and 1 to true when assigning values to bool
> variables. Inspired by commit 3db1cd5c05f35fb43eb134df6f321de4e63141f2.
>
> The simplified semantic patch that find this problem is as
> follows (http://coccinelle.lip6.fr/):
>
> @@
> bool b;
> @@
> (
> -b = 0
> +b = false
> |
> -b = 1
> +b = true
> )
>
> Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
>
> ---
> net/bluetooth/l2cap_core.c | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
Patch has been applied to bluetooth-next. Thanks.
Gustavo
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 3/5] OMAPDSS: DISPC: Fix assignment of 0/1 to bool variables
2013-09-22 18:44 [PATCH 1/5] scsi: Fix assignment of 0/1 to bool variables Peter Senna Tschudin
2013-09-22 18:44 ` [PATCH 2/5] Bluetooth: " Peter Senna Tschudin
@ 2013-09-22 18:44 ` Peter Senna Tschudin
2013-09-23 7:37 ` Tomi Valkeinen
2013-09-22 18:44 ` [PATCH 4/5] ALSA: " Peter Senna Tschudin
` (2 subsequent siblings)
4 siblings, 1 reply; 10+ messages in thread
From: Peter Senna Tschudin @ 2013-09-22 18:44 UTC (permalink / raw)
To: tomi.valkeinen
Cc: plagnioj, archit, linux-omap, linux-fbdev, linux-kernel,
kernel-janitors, Peter Senna Tschudin
Convert 0 to false and 1 to true when assigning values to bool
variables. Inspired by commit 3db1cd5c05f35fb43eb134df6f321de4e63141f2.
The simplified semantic patch that find this problem is as
follows (http://coccinelle.lip6.fr/):
@@
bool b;
@@
(
-b = 0
+b = false
|
-b = 1
+b = true
)
Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
---
drivers/video/omap2/dss/dispc.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff -u -p a/drivers/video/omap2/dss/dispc.c b/drivers/video/omap2/dss/dispc.c
--- a/drivers/video/omap2/dss/dispc.c
+++ b/drivers/video/omap2/dss/dispc.c
@@ -2352,7 +2352,7 @@ int dispc_ovl_check(enum omap_plane plan
{
enum omap_overlay_caps caps = dss_feat_get_overlay_caps(plane);
bool five_taps = true;
- bool fieldmode = 0;
+ bool fieldmode = false;
u16 in_height = oi->height;
u16 in_width = oi->width;
bool ilace = timings->interlace;
@@ -2365,7 +2365,7 @@ int dispc_ovl_check(enum omap_plane plan
out_height = oi->out_height == 0 ? oi->height : oi->out_height;
if (ilace && oi->height == out_height)
- fieldmode = 1;
+ fieldmode = true;
if (ilace) {
if (fieldmode)
@@ -2396,7 +2396,7 @@ static int dispc_ovl_setup_common(enum o
bool mem_to_mem)
{
bool five_taps = true;
- bool fieldmode = 0;
+ bool fieldmode = false;
int r, cconv = 0;
unsigned offset0, offset1;
s32 row_inc;
@@ -2417,7 +2417,7 @@ static int dispc_ovl_setup_common(enum o
out_height = out_height == 0 ? height : out_height;
if (ilace && height == out_height)
- fieldmode = 1;
+ fieldmode = true;
if (ilace) {
if (fieldmode)
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH 3/5] OMAPDSS: DISPC: Fix assignment of 0/1 to bool variables
2013-09-22 18:44 ` [PATCH 3/5] OMAPDSS: DISPC: " Peter Senna Tschudin
@ 2013-09-23 7:37 ` Tomi Valkeinen
0 siblings, 0 replies; 10+ messages in thread
From: Tomi Valkeinen @ 2013-09-23 7:37 UTC (permalink / raw)
To: Peter Senna Tschudin
Cc: plagnioj, archit, linux-omap, linux-fbdev, linux-kernel,
kernel-janitors
[-- Attachment #1: Type: text/plain, Size: 449 bytes --]
Hi,
On 22/09/13 21:44, Peter Senna Tschudin wrote:
> Convert 0 to false and 1 to true when assigning values to bool
> variables. Inspired by commit 3db1cd5c05f35fb43eb134df6f321de4e63141f2.
>
> The simplified semantic patch that find this problem is as
> follows (http://coccinelle.lip6.fr/):
Thanks, applied for 3.13. Although "fix" and "problem" are perhaps a bit
too strong words, as this is just a cosmetic change =).
Tomi
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 901 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 4/5] ALSA: Fix assignment of 0/1 to bool variables
2013-09-22 18:44 [PATCH 1/5] scsi: Fix assignment of 0/1 to bool variables Peter Senna Tschudin
2013-09-22 18:44 ` [PATCH 2/5] Bluetooth: " Peter Senna Tschudin
2013-09-22 18:44 ` [PATCH 3/5] OMAPDSS: DISPC: " Peter Senna Tschudin
@ 2013-09-22 18:44 ` Peter Senna Tschudin
2013-09-26 7:57 ` Takashi Iwai
2013-09-22 18:44 ` [PATCH 5/5] Staging: crystalhd: " Peter Senna Tschudin
2013-09-23 0:29 ` [PATCH 1/5] scsi: " Joe Perches
4 siblings, 1 reply; 10+ messages in thread
From: Peter Senna Tschudin @ 2013-09-22 18:44 UTC (permalink / raw)
To: perex
Cc: tiwai, wfp5p, zonque, eldad, david.henningsson, hegge,
linux-kernel, kernel-janitors, Peter Senna Tschudin, alsa-devel
Convert 0 to false and 1 to true when assigning values to bool
variables. Inspired by commit 3db1cd5c05f35fb43eb134df6f321de4e63141f2.
The simplified semantic patch that find this problem is as
follows (http://coccinelle.lip6.fr/):
@@
bool b;
@@
(
-b = 0
+b = false
|
-b = 1
+b = true
)
Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
---
sound/pci/azt3328.c | 14 +++++++-------
sound/usb/mixer.c | 4 ++--
2 files changed, 9 insertions(+), 9 deletions(-)
diff -u -p a/sound/usb/mixer.c b/sound/usb/mixer.c
--- a/sound/usb/mixer.c
+++ b/sound/usb/mixer.c
@@ -1151,14 +1151,14 @@ static void check_no_speaker_on_headset(
const char *names_to_check[] = {
"Headset", "headset", "Headphone", "headphone", NULL};
const char **s;
- bool found = 0;
+ bool found = false;
if (strcmp("Speaker", kctl->id.name))
return;
for (s = names_to_check; *s; s++)
if (strstr(card->shortname, *s)) {
- found = 1;
+ found = true;
break;
}
diff -u -p a/sound/pci/azt3328.c b/sound/pci/azt3328.c
--- a/sound/pci/azt3328.c
+++ b/sound/pci/azt3328.c
@@ -715,14 +715,14 @@ snd_azf3328_mixer_ac97_read(struct snd_a
const struct snd_azf3328 *chip = ac97->private_data;
unsigned short reg_azf = snd_azf3328_mixer_ac97_map_reg_idx(reg_ac97);
unsigned short reg_val = 0;
- bool unsupported = 0;
+ bool unsupported = false;
snd_azf3328_dbgmixer(
"snd_azf3328_mixer_ac97_read reg_ac97 %u\n",
reg_ac97
);
if (reg_azf & AZF_AC97_REG_UNSUPPORTED)
- unsupported = 1;
+ unsupported = true;
else {
if (reg_azf & AZF_AC97_REG_REAL_IO_READ)
reg_val = snd_azf3328_mixer_inw(chip,
@@ -759,7 +759,7 @@ snd_azf3328_mixer_ac97_read(struct snd_a
reg_val = azf_emulated_ac97_vendor_id & 0xffff;
break;
default:
- unsupported = 1;
+ unsupported = true;
break;
}
}
@@ -776,14 +776,14 @@ snd_azf3328_mixer_ac97_write(struct snd_
{
const struct snd_azf3328 *chip = ac97->private_data;
unsigned short reg_azf = snd_azf3328_mixer_ac97_map_reg_idx(reg_ac97);
- bool unsupported = 0;
+ bool unsupported = false;
snd_azf3328_dbgmixer(
"snd_azf3328_mixer_ac97_write reg_ac97 %u val %u\n",
reg_ac97, val
);
if (reg_azf & AZF_AC97_REG_UNSUPPORTED)
- unsupported = 1;
+ unsupported = true;
else {
if (reg_azf & AZF_AC97_REG_REAL_IO_WRITE)
snd_azf3328_mixer_outw(
@@ -808,7 +808,7 @@ snd_azf3328_mixer_ac97_write(struct snd_
*/
break;
default:
- unsupported = 1;
+ unsupported = true;
break;
}
}
@@ -1559,7 +1559,7 @@ snd_azf3328_pcm_trigger(struct snd_pcm_s
struct snd_azf3328_codec_data *codec = runtime->private_data;
int result = 0;
u16 flags1;
- bool previously_muted = 0;
+ bool previously_muted = false;
bool is_main_mixer_playback_codec = (AZF_CODEC_PLAYBACK == codec->type);
snd_azf3328_dbgcalls("snd_azf3328_pcm_trigger cmd %d\n", cmd);
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH 4/5] ALSA: Fix assignment of 0/1 to bool variables
2013-09-22 18:44 ` [PATCH 4/5] ALSA: " Peter Senna Tschudin
@ 2013-09-26 7:57 ` Takashi Iwai
0 siblings, 0 replies; 10+ messages in thread
From: Takashi Iwai @ 2013-09-26 7:57 UTC (permalink / raw)
To: Peter Senna Tschudin
Cc: perex, wfp5p, zonque, eldad, david.henningsson, hegge,
linux-kernel, kernel-janitors, alsa-devel
At Sun, 22 Sep 2013 20:44:12 +0200,
Peter Senna Tschudin wrote:
>
> Convert 0 to false and 1 to true when assigning values to bool
> variables. Inspired by commit 3db1cd5c05f35fb43eb134df6f321de4e63141f2.
>
> The simplified semantic patch that find this problem is as
> follows (http://coccinelle.lip6.fr/):
>
> @@
> bool b;
> @@
> (
> -b = 0
> +b = false
> |
> -b = 1
> +b = true
> )
>
> Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
Thanks, applied now.
Takashi
>
> ---
> sound/pci/azt3328.c | 14 +++++++-------
> sound/usb/mixer.c | 4 ++--
> 2 files changed, 9 insertions(+), 9 deletions(-)
>
> diff -u -p a/sound/usb/mixer.c b/sound/usb/mixer.c
> --- a/sound/usb/mixer.c
> +++ b/sound/usb/mixer.c
> @@ -1151,14 +1151,14 @@ static void check_no_speaker_on_headset(
> const char *names_to_check[] = {
> "Headset", "headset", "Headphone", "headphone", NULL};
> const char **s;
> - bool found = 0;
> + bool found = false;
>
> if (strcmp("Speaker", kctl->id.name))
> return;
>
> for (s = names_to_check; *s; s++)
> if (strstr(card->shortname, *s)) {
> - found = 1;
> + found = true;
> break;
> }
>
> diff -u -p a/sound/pci/azt3328.c b/sound/pci/azt3328.c
> --- a/sound/pci/azt3328.c
> +++ b/sound/pci/azt3328.c
> @@ -715,14 +715,14 @@ snd_azf3328_mixer_ac97_read(struct snd_a
> const struct snd_azf3328 *chip = ac97->private_data;
> unsigned short reg_azf = snd_azf3328_mixer_ac97_map_reg_idx(reg_ac97);
> unsigned short reg_val = 0;
> - bool unsupported = 0;
> + bool unsupported = false;
>
> snd_azf3328_dbgmixer(
> "snd_azf3328_mixer_ac97_read reg_ac97 %u\n",
> reg_ac97
> );
> if (reg_azf & AZF_AC97_REG_UNSUPPORTED)
> - unsupported = 1;
> + unsupported = true;
> else {
> if (reg_azf & AZF_AC97_REG_REAL_IO_READ)
> reg_val = snd_azf3328_mixer_inw(chip,
> @@ -759,7 +759,7 @@ snd_azf3328_mixer_ac97_read(struct snd_a
> reg_val = azf_emulated_ac97_vendor_id & 0xffff;
> break;
> default:
> - unsupported = 1;
> + unsupported = true;
> break;
> }
> }
> @@ -776,14 +776,14 @@ snd_azf3328_mixer_ac97_write(struct snd_
> {
> const struct snd_azf3328 *chip = ac97->private_data;
> unsigned short reg_azf = snd_azf3328_mixer_ac97_map_reg_idx(reg_ac97);
> - bool unsupported = 0;
> + bool unsupported = false;
>
> snd_azf3328_dbgmixer(
> "snd_azf3328_mixer_ac97_write reg_ac97 %u val %u\n",
> reg_ac97, val
> );
> if (reg_azf & AZF_AC97_REG_UNSUPPORTED)
> - unsupported = 1;
> + unsupported = true;
> else {
> if (reg_azf & AZF_AC97_REG_REAL_IO_WRITE)
> snd_azf3328_mixer_outw(
> @@ -808,7 +808,7 @@ snd_azf3328_mixer_ac97_write(struct snd_
> */
> break;
> default:
> - unsupported = 1;
> + unsupported = true;
> break;
> }
> }
> @@ -1559,7 +1559,7 @@ snd_azf3328_pcm_trigger(struct snd_pcm_s
> struct snd_azf3328_codec_data *codec = runtime->private_data;
> int result = 0;
> u16 flags1;
> - bool previously_muted = 0;
> + bool previously_muted = false;
> bool is_main_mixer_playback_codec = (AZF_CODEC_PLAYBACK == codec->type);
>
> snd_azf3328_dbgcalls("snd_azf3328_pcm_trigger cmd %d\n", cmd);
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 5/5] Staging: crystalhd: Fix assignment of 0/1 to bool variables
2013-09-22 18:44 [PATCH 1/5] scsi: Fix assignment of 0/1 to bool variables Peter Senna Tschudin
` (2 preceding siblings ...)
2013-09-22 18:44 ` [PATCH 4/5] ALSA: " Peter Senna Tschudin
@ 2013-09-22 18:44 ` Peter Senna Tschudin
2013-09-23 0:29 ` [PATCH 1/5] scsi: " Joe Perches
4 siblings, 0 replies; 10+ messages in thread
From: Peter Senna Tschudin @ 2013-09-22 18:44 UTC (permalink / raw)
To: nsankar
Cc: jarod, davilla, abraham.manu, gregkh, amarjargal16, devel,
linux-kernel, kernel-janitors, Peter Senna Tschudin
Convert 0 to false and 1 to true when assigning values to bool
variables. Inspired by commit 3db1cd5c05f35fb43eb134df6f321de4e63141f2.
The simplified semantic patch that find this problem is as
follows (http://coccinelle.lip6.fr/):
@@
bool b;
@@
(
-b = 0
+b = false
|
-b = 1
+b = true
)
Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
---
drivers/staging/crystalhd/crystalhd_hw.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff -u -p a/drivers/staging/crystalhd/crystalhd_hw.c b/drivers/staging/crystalhd/crystalhd_hw.c
--- a/drivers/staging/crystalhd/crystalhd_hw.c
+++ b/drivers/staging/crystalhd/crystalhd_hw.c
@@ -1517,7 +1517,7 @@ static void crystalhd_rx_isr(struct crys
uint32_t i, list_avail = 0;
enum BC_STATUS comp_sts = BC_STS_NO_DATA;
uint32_t y_err_sts, uv_err_sts, y_dn_sz = 0, uv_dn_sz = 0;
- bool ret = 0;
+ bool ret = false;
if (!hw) {
BCMLOG_ERR("Invalid Arguments\n");
@@ -1852,7 +1852,7 @@ bool crystalhd_hw_interrupt(struct cryst
{
uint32_t intr_sts = 0;
uint32_t deco_intr = 0;
- bool rc = 0;
+ bool rc = false;
if (!adp || !hw->dev_started)
return rc;
@@ -1865,7 +1865,7 @@ bool crystalhd_hw_interrupt(struct cryst
if (intr_sts) {
/* let system know we processed interrupt..*/
- rc = 1;
+ rc = true;
hw->stats.dev_interrupts++;
}
@@ -1886,7 +1886,7 @@ bool crystalhd_hw_interrupt(struct cryst
/* FIXME: jarod: No udelay? might this be
the real reason mini pci-e cards were stalling out? */
bc_dec_reg_wr(adp, Stream2Host_Intr_Sts, 0);
- rc = 1;
+ rc = true;
}
/* Rx interrupts */
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH 1/5] scsi: Fix assignment of 0/1 to bool variables
2013-09-22 18:44 [PATCH 1/5] scsi: Fix assignment of 0/1 to bool variables Peter Senna Tschudin
` (3 preceding siblings ...)
2013-09-22 18:44 ` [PATCH 5/5] Staging: crystalhd: " Peter Senna Tschudin
@ 2013-09-23 0:29 ` Joe Perches
4 siblings, 0 replies; 10+ messages in thread
From: Joe Perches @ 2013-09-23 0:29 UTC (permalink / raw)
To: Peter Senna Tschudin
Cc: JBottomley, naresh, davem, jj, arvindb, hare, michaelc,
vijay.chauhan, bvanassche, linux-scsi, linux-kernel,
kernel-janitors
On Sun, 2013-09-22 at 20:44 +0200, Peter Senna Tschudin wrote:
> Convert 0 to false and 1 to true when assigning values to bool
> variables. Inspired by commit 3db1cd5c05f35fb43eb134df6f321de4e63141f2.
[]
> @@
> bool b;
> @@
> -b = 1
> +b = true
You might also look for non-zero values other than 1
and convert those true too.
^ permalink raw reply [flat|nested] 10+ messages in thread