Hello Ashesh,
Right, bringing related logic close together is much better. But there
is a small difference in your version. Per discussion held on
pgadmin-support,
>> ...
>> vertical formatting shouldn't be applied where
>> number of parameters is less than two.
DP> Right.
So we expect:
CREATE OR REPLACE FUNCTION simple_function(param1 integer)
RETURNS integer AS
$BODY$
select 1;
$BODY$
LANGUAGE sql ...
but get:
CREATE OR REPLACE FUNCTION simple_function(
param1 integer)
RETURNS integer AS ...
I suggest changing line 518 from:
else if (multiline)
to
else if (multiline && argTypesArray.Count() > 1)
Thanks.
Tuesday, October 28, 2014, 2:35:50 PM, you wrote:
AV> Beautify the query shown in the SQL pane for the functions having multiple
AV> arguments.
AV> Reviewed by Sanket Mehta and small modification done by Ashesh Vashi.
AV> Branch
AV> ------
AV> master
AV> Details
AV> -------
AV>
http://git.postgresql.org/gitweb?p=pgadmin3.git;a=commitdiff;h=ea803ad68290bb506fd2148ee8ea8f4176e34197
AV> Author: J.F. Oster <jinfros...@mail.ru>
AV> Modified Files
AV> --------------
AV> CHANGELOG | 3 +++
AV> pgadmin/include/schema/pgFunction.h | 2 +-
AV> pgadmin/schema/pgFunction.cpp | 12 +++++++++---
AV> 3 files changed, 13 insertions(+), 4 deletions(-)
--
Best regards,
J.F.
--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers
