From: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
---|---|
To: | Roger Leigh <rleigh(at)codelibre(dot)net> |
Cc: | "Brad T(dot) Sliger" <brad(at)sliger(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Andrew Dunstan <andrew(at)dunslane(dot)net>, pgsql-hackers(at)postgresql(dot)org, Robert Haas <robertmhaas(at)gmail(dot)com>, Selena Deckelmann <selenamarie(at)gmail(dot)com>, Alvaro Herrera <alvherre(at)commandprompt(dot)com>, Roger Leigh <rleigh(at)debian(dot)org> |
Subject: | Re: Unicode UTF-8 table formatting for psql text output |
Date: | 2009-10-04 20:22:27 |
Message-ID: | 1254687747.13655.11.camel@vanquo.pezone.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
I have a comment on this bit:
> @@ -125,6 +128,17 @@ main(int argc, char *argv[])
>
> /* We rely on unmentioned fields of pset.popt to start out
> 0/false/NULL */
> pset.popt.topt.format = PRINT_ALIGNED;
> +
> + /* Default table style to plain ASCII */
> + pset.popt.topt.table_style = &asciiformat;
> +#if (defined(HAVE_LANGINFO_H) && defined(CODESET))
> + /* If a UTF-8 locale is available, switch to UTF-8 box drawing
> characters */
> + if (pg_strcasecmp(nl_langinfo(CODESET), "UTF-8") == 0 ||
> + pg_strcasecmp(nl_langinfo(CODESET), "utf8") == 0 ||
> + pg_strcasecmp(nl_langinfo(CODESET), "CP65001") == 0)
> + pset.popt.topt.table_style = &utf8format;
> +#endif
> +
> pset.popt.topt.border = 1;
> pset.popt.topt.pager = 1;
> pset.popt.topt.start_table = true;
Elsewhere in the psql code, notably in mbprint.c, we make the decision
on whether to apply certain Unicode-aware processing based on whether
the client encoding is UTF8. The same should be done here.
There is a patch somewhere in the pipeline that would automatically set
the psql client encoding to whatever the locale says, but until that is
done, the client encoding should be the sole setting that rules what
kind of character set processing is done on the client side.
From | Date | Subject | |
---|---|---|---|
Next Message | Josh Berkus | 2009-10-04 20:25:31 | Re: Rules: A Modest Proposal |
Previous Message | Jeff Janes | 2009-10-04 20:14:32 | Re: Lock Wait Statistics (next commitfest) |