Re: Patch for psql to avoid altering the PGresult strings

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Dennis Björklund <db(at)zigo(dot)dhs(dot)org>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: Patch for psql to avoid altering the PGresult strings
Date: 2003-06-24 03:33:46
Message-ID: 200306240333.h5O3XkU15754@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches


What should be done with this patch?

---------------------------------------------------------------------------

Dennis Bjrklund wrote:
> The mbvalidate() function was called on the PGresult strings, and it
> changes the strings (removes unknown characters).
>
> I've change the validation function to one that either returns a new
> string when needed or NULL when the string already was validating. The
> normal case is to have validating strings so in most cases no new strings
> are created.
>
> I have not included changes to avoid sending non validating strings to the
> server. It's not clear what is the best way to solve it. Maybe one should
> just do a stupid fix for 7.4 and validate the strings like above and just
> delete non validating strings. It is possible to solve it in a better way,
> but it's more complicated. We have 4 different charsets to think about in
> psql. The server, the client, the terminal and the message catalog (the
> last two usually matches, but not always). I would prefer to find a good
> solution instead of just patching one small problem after another.
>
> --
> /Dennis

Content-Description:

[ Attachment, skipping... ]

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

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2003-06-24 03:34:18 Re: Patch to be verbose about being unable to read ~/.pgpasss...
Previous Message Bruce Momjian 2003-06-24 03:31:01 Re: [HACKERS] Subtraction carry bug in xlog.c in 7.3 and 7.4