Re: fix of some issues with multi-line query editing - Mailing list pgsql-patches

From Sergey E. Koposov
Subject Re: fix of some issues with multi-line query editing
Date
Msg-id Pine.LNX.4.44.0603131139340.11667-100000@lnfm1.sai.msu.ru
Whole thread Raw
In response to Re: fix of some issues with multi-line query editing  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: fix of some issues with multi-line query editing
List pgsql-patches
On Sun, 12 Mar 2006, Tom Lane wrote:

> Bruce Momjian <pgman@candle.pha.pa.us> writes:
> >> 3) Fix the handling of commands with not closed brackets, quotes, double
> >> quotes. (now those commands are not splitted in parts...)
>
> Really?  CVS tip seems to treat a multiline command as a single history
> entry *unless* it contains newlines within quoted strings (including
> single, double, dollar quotes) or newlines within /* comments.


Bruce have NOT applied MY patch, which really contained the fixes of the
described problems... So, I'm waiting for it...

>
> I think this patch is seriously broken, and I don't agree with what it's
> trying to accomplish in the first place --- I still haven't found any
> cases where it's an improvement to pull back multiple lines as one
> history entry.  For example, if you made a mistake on the fifth line of
> a ten-line CREATE TABLE command, the current code makes it extremely
> inconvenient to fix that: you have to back-arrow or forward-arrow
> tediously over five lines, where before you could pull back one line at
> a time and just edir the line that needed fixing.

(you do always Ctrl+Left_arrow ... to do that faster...) I don't see
anything bad here, that's the normal way to do it...

Regards,

    Sergey

*****************************************************
Sergey E. Koposov
Max Planck Institute for Astronomy
Web: http://lnfm1.sai.msu.ru/~math
E-mail: math@sai.msu.ru


pgsql-patches by date:

Previous
From: ITAGAKI Takahiro
Date:
Subject: Re: [HACKERS] Automatic free space map filling
Next
From: Martijn van Oosterhout
Date:
Subject: Re: Proposed patch for error locations