From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | Oliver Elphick <olly(at)lfix(dot)co(dot)uk> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Bad macro in pg_wchar.h? |
Date: | 2003-06-02 18:59:31 |
Message-ID: | 200306021859.h52IxVQ10235@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Patch applied. Thanks.
---------------------------------------------------------------------------
Oliver Elphick wrote:
> In src/include/mb/pg_wchar.h we have:
>
> #define PG_ENCODING_BE_LAST PG_ISO_8859_8
> #define PG_ENCODING_FE_LAST PG_WIN1256
>
> but the last client encoding in the enum list is actually PG_GB18030 and
> it seems that
>
> #define PG_ENCODING_IS_CLIEN_ONLY(_enc) \
> (((_enc) > PG_ENCODING_BE_LAST && (_enc) <= PG_ENCODING_FE_LAST)
>
> can never be true.
>
> I think the define should read
> #define PG_ENCODING_FE_LAST PG_GB18030
>
> On the other hand, perhaps no-one cares, because
> PG_ENCODING_IS_CLIEN_ONLY is never used.
>
> --
> Oliver Elphick Oliver(dot)Elphick(at)lfix(dot)co(dot)uk
> Isle of Wight, UK http://www.lfix.co.uk/oliver
> GPG: 1024D/3E1D0C1C: CA12 09E0 E8D5 8870 5839 932A 614D 4C34 3E1D 0C1C
> ========================================
> "O come, let us worship and bow down; let us kneel
> before the LORD our maker." Psalms 95:6
>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 5: Have you checked our extensive FAQ?
>
> http://www.postgresql.org/docs/faqs/FAQ.html
>
--
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
Attachment | Content-Type | Size |
---|---|---|
unknown_filename | text/plain | 723 bytes |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2003-06-02 19:02:51 | Re: initdb problems handling super username with spaces. |
Previous Message | Bruce Momjian | 2003-06-02 18:57:49 | Re: Postgres config file: autocommit = off |