Re: client side syntax error position (v3) - Mailing list pgsql-patches

From Fabien COELHO
Subject Re: client side syntax error position (v3)
Date
Msg-id Pine.LNX.4.58.0403151052450.3683@sablons.cri.ensmp.fr
Whole thread Raw
In response to Re: client side syntax error position (v3)  (Fabien COELHO <coelho@cri.ensmp.fr>)
Responses Re: client side syntax error position (v3)
List pgsql-patches
Dear Tom,

> >         scroffset += 1;        /* XXX fix me when we have screen width info */
> >
> > with calls to a get-the-screen-width-of-this-character subroutine should
> > do the job.
>
> I also looked into it, and it is also a little bit more complex, as the
> extract width must be though from the terminal perspective. Thus the
> truncation part must take into account the terminal lengths. I think it
> would require an additionnal array to store character to terminal column
> mapping that would be used when truncating. I'll do that as soon as the
> needed routines are there for terminal lengths, and submit a new patch.

Well, I finally noticed that you did all that already;-) So I won't
have anything to do. Good;-)

I should think and check before writing...

Also you improved/simplified the code for the better.

I've seen the gettext stuff and buffer data. I was wondering how the
localisation magic could work, it is quite simpler this way. Also you
changed the palloc/pfree stuff to pg_malloc/free. I'll have to check pg
memory management...

Thanks again, have a nice day,

--
Fabien Coelho - coelho@cri.ensmp.fr

pgsql-patches by date:

Previous
From: Fabien COELHO
Date:
Subject: Re: client side syntax error position (v3)
Next
From: Andrew Dunstan
Date:
Subject: default psql to localhost on platforms without unix domain sockets