Re: patch for ECPG (BUG #2956: ECPG does not treat multibyte characters correctly.) - Mailing list pgsql-patches

From Michael Meskes
Subject Re: patch for ECPG (BUG #2956: ECPG does not treat multibyte characters correctly.)
Date
Msg-id 20070211151604.GA30346@feivel.credativ.de
Whole thread Raw
In response to patch for ECPG (BUG #2956: ECPG does not treat multibyte characters correctly.)  (原田登志 <harada.toshi@oss.ntt.co.jp>)
Responses Re: patch for ECPG (BUG #2956: ECPG does not treat multibyte characters correctly.)  (ITAGAKI Takahiro <itagaki.takahiro@oss.ntt.co.jp>)
List pgsql-patches
On Fri, Feb 09, 2007 at 05:29:21PM +0900, 原田登志 wrote:
> I found bug in ecpg concerning processing of the multi-byte character-code.
> I reported as bug#2956 before.

Thanks for report and patch.

> Attached is the patch to fix this problem.

This patch is against 8.1, right? I had to apply it manually to HEAD, so
I might have made some error here.

Also I don't see why we have to add the connection to all thos function
calls, just to make sure we get an error message logged in the
connection struct that apparently isn't evaluated anyway. Please correct
me if I'm wrong on this one.

I'm just committing the changes to CVS but only to HEAD because I cannot
check if my changes broke something. The sources work fine on my system
and the regression tests pass without a problem. But then I do not have
a setup similar to yours. Could you please test this?

Michael
--
Michael Meskes
Email: Michael at Fam-Meskes dot De, Michael at Meskes dot (De|Com|Net|Org)
ICQ: 179140304, AIM/Yahoo: michaelmeskes, Jabber: meskes@jabber.org
Go SF 49ers! Go Rhein Fire! Use Debian GNU/Linux! Use PostgreSQL!

pgsql-patches by date:

Previous
From: Jeremy Drake
Date:
Subject: Re: patch adding new regexp functions
Next
From: "Pavel Stehule"
Date:
Subject: Table function support