On 3/10/16 9:51 AM, Tom Lane wrote:
> The patch is evidently modeled on errhidestmt and errhidectx, which are
> making the same assumption for their fields.
>
> I wonder whether, instead of continuing to proliferate random bool fields
> in struct ErrorData, we oughta replace them all with an "int flags" field.
> That's probably material for a separate patch though.
There are currently six boolean flags (if you include my new one) that
all relate to visibility in one way or another. Combining them into a
"flags" field makes sense to me.
I'll submit a proposal to hackers after 9.6 to make this change.
--
-David
david@pgmasters.net