Re: boolean in C - Mailing list pgsql-hackers

From Grzegorz Jaskiewicz
Subject Re: boolean in C
Date
Msg-id D1C736DE-E48F-418B-8D41-DFAF9F299C82@pointblue.com.pl
Whole thread Raw
In response to Re: boolean in C  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: boolean in C
List pgsql-hackers
On 16 Jul 2009, at 15:17, Tom Lane wrote:

> Grzegorz Jaskiewicz <gj@pointblue.com.pl> writes:
>
> That's hardly going to improve readability for anyone.  Also, it will
> flat out not work for the catalog struct declarations.  When we say
> "bool relhasindex;" the compiler had better think that that's a
> one-byte field.

Sure, but I would certainly hope, there's not too many places where  
you actually convert it from disc representation, to internal and vice  
versa.

>
>> And it is pretty annoying, when your product also has its own BOOLean
>> defined...
>
> IOW you're not using stdbool either?

Well, saying that I don't is quite an overstatement. It was decided  
long before I started working for that customer, and is full of  
problems like that. But still,
it would be nice for postgresql to at least not cause problems like  
that. Having said that, I will probably fix it on customer's side, but  
I wanted to see if you guys will be happy with patch that changes that  
in postgresql.

thanks .



pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: Mostly Harmless: c++bookends - patch 2 of 4
Next
From: Tom Lane
Date:
Subject: Re: Review remove {join,from}_collapse_limit, add enable_join_ordering