Re: wrapping in extended mode doesn't work well with default pager

Поиск

Список

Период

Сортировка

От	Pavel Stehule
Тема	Re: wrapping in extended mode doesn't work well with default pager
Дата	14 мая 2014 г. 09:56:08
Msg-id	CAFj8pRCurBefFcS0sxvq_e3XVUsMWtBPd3wPX2HuUQhsnZRGhA@mail.gmail.com обсуждение исходный текст
Ответ на	Re: wrapping in extended mode doesn't work well with default pager (Pavel Stehule <pavel.stehule@gmail.com>)
Ответы	Re: wrapping in extended mode doesn't work well with default pager
Список	pgsql-hackers

Дерево обсуждения

sorry

there is still small issue

I have a plpgsql function:

CREATE OR REPLACE FUNCTION public.foo_update_trg()
RETURNS trigger
LANGUAGE plpgsql
AS $function$
DECLARE t text;
BEGIN
EXECUTE format('SELECT $1.%I', TG_ARGV[0]) INTO t USING old;
RAISE NOTICE 'original value of "%" is "%"', TG_ARGV[0], t;
RETURN NULL;
END;
$function$

Default expanded view of select * from pg_proc where proname = 'foo_update_trg'; is little bit broken (screenshoot 1)

After wrap mode, it add useless new line into source code (screenshoot 2)

but border2 fixes it (screenshots 3)

Regards

Pavel

2014-05-14 8:32 GMT+02:00 Pavel Stehule <pavel.stehule@gmail.com>:

Hello

With this patch it works perfect

Thank you

Regards

Pavel

2014-05-13 21:33 GMT+02:00 Sergey Muraviov <sergey.k.muraviov@gmail.com>:

Please check this patch.

2014-05-12 22:56 GMT+04:00 Sergey Muraviov <sergey.k.muraviov@gmail.com>:

Hi.

I'll try to fix it tomorrow.

2014-05-12 18:42 GMT+04:00 Tom Lane <tgl@sss.pgh.pa.us>:

Greg Stark <stark@mit.edu> writes:
> On Mon, May 12, 2014 at 2:12 PM, Greg Stark <stark@mit.edu> wrote:
>> Hm, there was an off by one error earlier in some cases, maybe we
>> fixed it by breaking other case. Will investigate.

> Those spaces are coming from the ascii wrapping indicators. i.e. the periods in:

Ah. I wonder whether anyone will complain that the format changed?

> Apparently we used to print those with border=1 in normal mode but in
> expanded mode we left out the space for those on the outermost edges
> since there was no need for them. If we put them in for wrapped mode
> then we'll be inconsistent if we don't for nonwrapped mode though. And
> if we don't put them in for wrapped mode then there's no way to
> indicate wrapping versus newlines.

Barring anyone complaining that the format changed, I'd say the issue
is not that you added them but that the accounting for line length
fails to include them.

regards, tom lane

--
Best regards,
Sergey Muraviov

--
Best regards,
Sergey MuraviovH