Re: [patch 1/1] consolidate TRUE and FALSE

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Xavier Bestel <[email protected]> wrote:
>
> On Thu, 16 Mar 2006 13:26:39 -0800
> Andrew Morton <[email protected]> wrote:
> 
> > C99 does have boolean support, so the proper thing to do is to start
> > using it - implement stdbool.h, fix up fallout, start fixing subsystems. 
> > Given that, and as Greg has fixed up this particular build error I'll drop
> > the patch.
> 
> Isn't there a runtime cost converting all "non-false" values to a unique "true" (i.e. converting non-zero values to one) ?

Yes, there will be.  If people do wrong things.

> I mean:
> 
> bool res = strcmp(string, "whatever");
> if(res)
> 	something_else();

There's an implicit conversion from integer (-1, 0, 1) to boolean there. 
It _should_ require a typecast or, better,

	bool res = (strcmp(...) != 0);

But it won't require that - the compiler will just accept it..  A new `gcc
--implement-bools-properly' or perhaps sparse should warn about the above.

(Hopefully the compiler would have the brains to optimise it all away in
this specific example).

But yeah, it's always possible to deoptimise your code.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux