RE: [HACKERS] create database doesn't work well in MULTIBYTE mode

> -----Original Message-----
> From: Bruce Momjian [mailto:pgman(at)candle(dot)pha(dot)pa(dot)us]
> Sent: Thursday, February 17, 2000 2:41 PM
>
> > I have a crash while creating regression database in pararell regression
> > test.
> > Seems it's due to the following change.
> >
> > @@ -2638,7 +2705,14 @@
> > n->dbname = $3;
> > n->dbpath = $5;
> > #ifdef MULTIBYTE
> > - n->encoding = $6;
> > + if ($6 != NULL) {
> > + n->encoding =
> pg_char_to_encoding($6);
> > + if (n->encoding < 0) {
> > + elog(ERROR,
> "Encoding name '%s' is invalid", $6);
> > + }
> > + } else {
> > + n->encoding =
> GetTemplateEncoding();
> > + }
> > #else
> > n->encoding = 0;
> > #endif
> >
> > Why ?
> > $6 is an ival not an str.
>
> Not sure what to recommend here, but you clearly have found a problem.
> Try it as a string, and if that works, patch it.
>

$6 is already converted from string to ival in another place.
It seems to me that this change is unnecessary.
I don't understand why this was changed recently.

Regards.

Hiroshi Inoue
Inoue(at)tpf(dot)co(dot)jp