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

From Tom Lane
Subject Re: fix of some issues with multi-line query editing
Date
Msg-id 26953.1142223875@sss.pgh.pa.us
Whole thread Raw
In response to Re: fix of some issues with multi-line query editing  (Bruce Momjian <pgman@candle.pha.pa.us>)
Responses Re: fix of some issues with multi-line query editing
Re: fix of some issues with multi-line query editing
List pgsql-patches
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.

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.

I'm still a vote to revert the patch altogether.  But there is no theory
under which the current behavior is sane.

            regards, tom lane

pgsql-patches by date:

Previous
From: Christopher Kings-Lynne
Date:
Subject: Re: [HACKERS] pg_freespacemap question
Next
From: Mark Kirkwood
Date:
Subject: Re: [HACKERS] pg_freespacemap question