Re: [PATCH] Formatting patch for psql

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Martijn van Oosterhout <kleptog(at)svana(dot)org>
Cc: Peter Eisentraut <peter_e(at)gmx(dot)net>, pgsql-patches(at)postgresql(dot)org
Subject: Re: [PATCH] Formatting patch for psql
Date: 2005-11-25 14:58:21
Message-ID: 22120.1132930701@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

Martijn van Oosterhout <kleptog(at)svana(dot)org> writes:
> ... There is a fair
> bit of encoding code in there already but the information you need here
> is specifically: is this char a control character and in particular, is
> it a newline.

The appropriate test for that is just "ch == '\n'", in all the encodings
we support (although you do have to be careful that ch isn't a non-first
byte of a multibyte character). If that's all you need then inventing
a bunch of additional infrastructure is really inappropriate.

regards, tom lane

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Pavel Stehule 2005-11-25 16:01:35 is not distinct from
Previous Message Martijn van Oosterhout 2005-11-25 10:42:26 Re: [PATCH] Formatting patch for psql