linux-kernel - Re: [PATCH] Staging: vt6656: Change the define FALSE and TRUE to the lower case variant.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Tue, 13 Jul 2010 11:36:12 +0200
From:	Roel Van Nyen <roel.vannyen@...il.com>
To:	Bernd Petrovitsch <bernd@...rovitsch.priv.at>
Cc:	Greg Kroah-Hartman <gregkh@...e.de>,
	Forest Bond <forest@...ttletooquiet.net>,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] Staging: vt6656: Change the define FALSE and TRUE to the 
	lower case variant.

2010/7/13 Bernd Petrovitsch <bernd@...rovitsch.priv.at>
>
> Hi!
>
> On Mon, 2010-07-12 at 23:26 +0200, Roel Van Nyen wrote:
> [...]
> > @@ -178,7 +178,7 @@ vMgrDecodeBeacon(
> >              break;
> >          case WLAN_EID_RSN_WPA:
> >              if (pFrame->pRSNWPA == NULL) {
> > -                if (WPAb_Is_RSN((PWLAN_IE_RSN_EXT)pItem) == TRUE)
> > +                if (WPAb_Is_RSN((PWLAN_IE_RSN_EXT)pItem) == true)
> >                      pFrame->pRSNWPA = (PWLAN_IE_RSN_EXT)pItem;
> >              }
> >              break;
>
> Am I the only one who thinks that this should better be
> ----  snip  ----
> @@ -178,7 +178,7 @@ vMgrDecodeBeacon(
>             break;
>         case WLAN_EID_RSN_WPA:
>             if (pFrame->pRSNWPA == NULL) {
> -                if (WPAb_Is_RSN((PWLAN_IE_RSN_EXT)pItem) == TRUE)
> +                if (WPAb_Is_RSN((PWLAN_IE_RSN_EXT)pItem))
>                     pFrame->pRSNWPA = (PWLAN_IE_RSN_EXT)pItem;
>             }
>             break;
> ----  snip  ----
> - if only for readability and understandability?
>
>        Bernd
> --
> Bernd Petrovitsch                  Email : bernd@...rovitsch.priv.at
>                     LUGA : http://www.luga.at
>

Hi!

I think from a readability and understandability standpoint == true is
very clear, anyone who reads it will explicitly understand the
meaning.
The goal of my patch is to remove the defines from the ttype.h so that
at some time ttype.h can be removed.

Any more pros or cons for the == true ?
Roel
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/