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

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

 



On Thu, 2006-03-16 at 02:42 -0800, Andrew Morton wrote:
> Anton Altaparmakov <[email protected]> wrote:
> >
> > > Various places are doing things like
> >  > 
> >  > typedef {
> >  > 	FALSE,
> >  > 	TRUE
> >  > } my_fave_name_for_a_bool;
> >  > 
> >  > These are converted to
> >  > 
> >  > typedef int my_fave_name_for_a_bool;
> > 
> >  Given that the kernel now requires gcc 3.2 or later, that already includes 
> >  a native boolean type (_Bool)?
> 
> It does?
> 
> Is it any good?
> 
> bix:/home/akpm> cat t.c
> void foo()
> {
> 	_Bool b = 1;
> 
> 	b += 2;
> }
> bix:/home/akpm> gcc -O -Wall -c t.c
> bix:/home/akpm> 
> 
> Sigh.
> 

If you were to read the value of b after "b += 2", you'd find that b is
still 1.

Also note that _Bool is a byte, so anything that exposed its own custom
boolean type to userspace needs to be carefully updated.

-- 
Nicholas Miell <[email protected]>

-
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