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 -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers