On Wed, Sep 14, 2016 at 3:30 PM, Dave Page <dpage@pgadmin.org> wrote:
Thanks - committed with a couple of minor formatting tweaks:
- Remove the leading space before arguments (e.g. s/, /\r\t/ instead of s/,/\r\t/) - Put the first argument on a new line as well.
It seems fine except the case when there are no arguments, the closing bracket ends in next line. Is it ok?
Please double-check I didn't break anything that I've missed!
It doesn't break anything
.
On Tue, Sep 13, 2016 at 12:41 PM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote: > Hi > > Patch with following changes: > > 1) In SQL pane, the signature in comments had arguments with name and type, > Now it will show arguments with type only. > > 2) There is no indent+newline between arguments (there isn't even a space). > - Fixed > > 3) The default value for the last argument is omitted. - Fixed > Last argument is omitted because the value for the last argument we got from > table is empty. > Now we form function signature in the get_definition.sql using > 'pg_catalog.pg_get_function_arguments' method. > > Above changes will reflect in Functions, Procedures & Trigger function > Nodes. > > Please review attached patch. > > Thanks, > Surinder Kumar > >