"Sergey E. Koposov" <[EMAIL PROTECTED]> writes:
> On Wed, 7 Dec 2005, Andrew Dunstan wrote:
>> A zero byte is probably a pretty bad choice. Some other low valued byte 
>> (e.g. \x01 ) would probably work better.

> Currently I replace '\n' with the '\x01' as Andrew suggested.

Won't this get confused by some of the Far Eastern encodings we support?
The zero-byte approach is at least proof against that.  But what we need
to ask is whether we can expect readline to cope with either.

The patch *looks* pretty ugly: random insertions of blank space,
general failure to conform to the project's code layout conventions,
etc.  (Some of this would get cleaned up by pgindent, but I'm not sure
how much.)  Also I get the impression that the patch enforces a lot of
history maintenance overhead even in the non-USE_READLINE case, which is
surely useless.

                        regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 5: don't forget to increase your free space map settings

Reply via email to