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.
Please double-check I didn't break anything that I've missed! On Tue, Sep 13, 2016 at 12:41 PM, Surinder Kumar <surinder.ku...@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 > > > -- > Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) > To make changes to your subscription: > http://www.postgresql.org/mailpref/pgadmin-hackers > -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgadmin-hackers
