2014-03-14 12:02 GMT+01:00 Marko Tiikkaja <ma...@joh.to>: > 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' >> > +1
> plpgsql.compile_errors = 'list' default = 'none' >> >> Only possible values in 9.4 are 'shadow', 'all', 'none' >> > what is compile_errors ? We don't allow to ignore any error now. > > I'm fine with this. I'm starting to think that runtime warnings are a bad > idea anyway. > +1 Pavel > > > Regards, > Marko Tiikkaja >