Re: Patch for pl/tcl Tcl_ExternalToUtf and Tcl_UtfToExternal

From: Reinhard Max <max(at)suse(dot)de>
To: Vsevolod Lobko <seva(at)sevasoft(dot)kiev(dot)ua>
Cc: Peter Eisentraut <peter_e(at)gmx(dot)net>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-patches <pgsql-patches(at)postgresql(dot)org>, Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Subject: Re: Patch for pl/tcl Tcl_ExternalToUtf and Tcl_UtfToExternal
Date: 2001-09-06 11:12:32
Message-ID: Pine.LNX.4.33.0109061253420.14536-100000@wotan.suse.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

On Thu, 6 Sep 2001, Vsevolod Lobko wrote:

> > For a third proposal which is IMHO the cleanest one so far, please
> > have a look at the mail, I've sent to Bruce and psql-bugs earlier
> > today. In short: PostgreSQL should be compiled with --enable-multibyte
> > and --enable-unicode-convertion and the Tcl interfaces (PgTcl and
> > PL/Tcl) should be changed to set their client encoding to UNICODE if
> > the Tcl version requires UTF-8 strings.
>
> third proposal relates to pgtcl clients only, patch deals with pl/tcl.
> So it's unrelated problems.

Not really. There is a similar patch proposed for PgTcl and I think
the problem should be resolved having both libraries in mind to get a
consistent behaviour on whatever side - client or server - Tcl is
used.

BTW, the proposal in your other mail to use pg_do_encoding_conversion
for PL/Tcl seems to be in the sense than my proposal to use UNICODE as
client encoding in PgTcl and it would also need PostgreSQL to be
compiled with --enable-multibyte and --enable-unicode-convertion.

So this seems to be the most clean and consistent solution:

- PostgreSQL requires to be compiled with --enable-multibyte
and --enable-unicode-convertion if it ought to work correctly
with Tcl/Tk >= 8.1 (client or server side).

- PL/Tcl needs to be changed to use pg_do_encoding_conversion
if it runs on a Tcl version >= 8.1 .

- PgTcl needs to be changed to set it's cliend encoding to UNICODE
if it runs on a Tcl version >= 8.1 .

Can we agree on that?

cu
Reinhard

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Vsevolod Lobko 2001-09-06 11:19:00 Re: Patch for pl/tcl Tcl_ExternalToUtf and Tcl_UtfToExternal
Previous Message Vsevolod Lobko 2001-09-06 10:52:43 Re: Patch for pl/tcl Tcl_ExternalToUtf and Tcl_UtfToExternal