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