Re: pg_do_encoding_conversion glitch

From: ITAGAKI Takahiro <itagaki(dot)takahiro(at)oss(dot)ntt(dot)co(dot)jp>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: pg_do_encoding_conversion glitch
Date: 2008-11-11 00:56:21
Message-ID: 20081111094111.7BDD.52131E4D@oss.ntt.co.jp
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:

> Do you have a proposal for a different API, or do you just want to
> improve the comment for the function? Bear in mind that a lot of the
> callers do know the string length, and so we shouldn't impose an
> unnecessary strlen() operation on those cases.

We already have the following comment, so I think a new comment is
not needed.
| In the case of no conversion, src is returned.

Since Assert() is not available in the case, developers should use
the function carefully after all. My patch might be fixed, too...
| "Solve a problem of LC_TIME of windows"
| http://archives.postgresql.org/message-id/20081104094301.7EE8.52131E4D@oss.ntt.co.jp

Regards,
---
ITAGAKI Takahiro
NTT Open Source Software Center

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2008-11-11 00:58:17 Re: Patch for ISO-8601-Interval Input and output.
Previous Message Bruce Momjian 2008-11-11 00:47:41 Re: pg_upgrade project status