Re: WITH CHECK OPTION for auto-updatable views - Mailing list pgsql-hackers

From Dean Rasheed
Subject Re: WITH CHECK OPTION for auto-updatable views
Date
Msg-id CAEZATCUUR9pmwLp83-PWz7GTV+zR8ovS2GQaNXQDGe38K_49jw@mail.gmail.com
Whole thread Raw
In response to Re: WITH CHECK OPTION for auto-updatable views  (Pavel Stehule <pavel.stehule@gmail.com>)
List pgsql-hackers
On 5 July 2013 07:02, Pavel Stehule <pavel.stehule@gmail.com> wrote:
> Hello
>
> I try to check this patch
>
> I have a problem with initdb after patching
>
> error
>
> initializing dependencies ... ok
> creating system views ... FATAL:  WITH CHECK OPTION is supported only
> on auto-updatable views
> STATEMENT:  /*
>
>
> I found missing initialization (strange, gcc doesn't raise warnings :( )
>
> +               bool            check_option;
> +               bool            security_barrier;
>

Ah, good catch. I was being careless there.

It turns out that although I compile with -Wall which implies
-Wuninitialized and -Wmaybe-uninitialized, those warnings are only
supported in optimised builds, which is why I didn't see it. So I've
learned something new today: always do an optimised build as well
during development.

Will fix. Thanks.

Regards,
Dean



pgsql-hackers by date:

Previous
From: Michael Meskes
Date:
Subject: Re: [9.3 bug fix] ECPG does not escape backslashes
Next
From: "MauMau"
Date:
Subject: Re: [9.3 bug fix] ECPG does not escape backslashes