Received: from localhost (unknown [200.46.204.183]) by mail.postgresql.org (Postfix) with ESMTP id 44D5B632364 for ; Sat, 7 Feb 2009 11:11:25 -0400 (AST) Received: from mail.postgresql.org ([200.46.204.86]) by localhost (mx1.hub.org [200.46.204.183]) (amavisd-maia, port 10024) with ESMTP id 55020-09 for ; Sat, 7 Feb 2009 11:11:21 -0400 (AST) X-Greylist: from auto-whitelisted by SQLgrey-1.7.6 Received: from momjian.us (momjian.us [70.90.9.53]) by mail.postgresql.org (Postfix) with ESMTP id 4C9BF632191 for ; Sat, 7 Feb 2009 11:11:07 -0400 (AST) Received: (from bruce@localhost) by momjian.us (8.11.6/8.11.6) id n17FB4t23514; Sat, 7 Feb 2009 10:11:04 -0500 (EST) From: Bruce Momjian Message-Id: <200902071511.n17FB4t23514@momjian.us> Subject: Re: [COMMITTERS] pgsql: Explicitly bind gettext to the correct encoding on Windows. In-Reply-To: <20643.1232666867@sss.pgh.pa.us> To: Tom Lane Date: Sat, 7 Feb 2009 10:11:04 -0500 (EST) CC: Magnus Hagander , pgsql-hackers@postgresql.org X-Mailer: ELM [version 2.4ME+ PL124 (25)] MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="US-ASCII" X-Virus-Scanned: Maia Mailguard 1.0.1 X-Spam-Status: No, hits=0 tagged_above=0 required=5 tests=none X-Spam-Level: X-Archive-Number: 200902/328 X-Sequence-Number: 133618 Tom Lane wrote: > mha@postgresql.org (Magnus Hagander) writes: > > Explicitly bind gettext to the correct encoding on Windows. > > I have a couple of objections to this patch. First, what happens if > it fails to find a matching table entry? (The existing answer is > "nothing", but that doesn't seem right.) Second and more critical, > it adds still another data structure that has to be maintained when > the list of encodings changes, and it doesn't even live in the same > file as any existing encoding-information table. > > What makes more sense to me is to add a table to encnames.c that > provides the gettext name of every encoding that we support. Would someone please comment on Tom's questions above. -- Bruce Momjian http://momjian.us EnterpriseDB http://enterprisedb.com + If your life is a hard drive, Christ can be your backup. +