Re: plpgsql.warn_shadow

On 3/14/14 10:56 AM, Simon Riggs wrote:
> The patch looks fine, apart from some non-guideline code formatting issues.

I'm not sure what you're referring to.  I thought it looked fine.

> Having looked at gcc and clang, I have a proposal for naming/API
>
> We just have two variables
>
>    plpgsql.compile_warnings = 'list'    default = 'none'
>    plpgsql.compile_errors = 'list'        default = 'none'
>
> Only possible values in 9.4 are 'shadow', 'all', 'none'

I'm fine with this.  I'm starting to think that runtime warnings are a 
bad idea anyway.


Regards,
Marko Tiikkaja