Re: [HACKERS] Spaces in psql output (Was: FW: PGBuildfarm member snake Branch HEAD Status changed)

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Martijn van Oosterhout <kleptog(at)svana(dot)org>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: [HACKERS] Spaces in psql output (Was: FW: PGBuildfarm member snake Branch HEAD Status changed)
Date: 2006-02-12 03:42:59
Message-ID: 7617.1139715779@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Martijn van Oosterhout <kleptog(at)svana(dot)org> writes:
> Well, the attached patch removes the padding on the last column,
> irrespective of the line. It will pad all the way to the end if the
> cell is empty due to one of the earlier columns being multiline.

Applied with some cosmetic changes. I've confirmed that all the
regression tests pass with "-w" removed from DIFFFLAGS ... which
was a good exercise, actually, because it exposed the fact that
the line-wrapping patch broke ReportSyntaxErrorPosition. I've put a
band-aid on that, though it might be worth trying to make it smarter
about control characters in general.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2006-02-12 03:55:59 Re: TODO Item - Add system view to show free space map
Previous Message Chris Browne 2006-02-12 01:57:47 Re: How to VACUUM this table? "998994633 estimated total rows"

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2006-02-12 03:46:58 Re: TODO item -- Improve psql's handling of multi-line
Previous Message Tom Lane 2006-02-12 03:39:44 Re: TODO item -- Improve psql's handling of multi-line