Bruce Momjian wrote:
> ! /* print a divider, middle columns only
> */
> ! if ((j + 1) % col_count)
> {
> ! if (opt_border == 0)
> ! fputc(' ', fout);
> ! /* first line for values? */
> ! else if (line_count == 0 &&
> col_line_count == 0)
> ! fputs(" | ", fout);
> ! /* next value is beyond height?
> */
> ! else if (line_count >=
> heights[j + 1])
> ! fputs(" ", fout);
> ! /* start of another newline
> string? */
> ! else if (col_line_count == 0)
> ! fputs(" : ", fout);
> ! else
> ! {
> ! /* Does the next column
> wrap to this line? */
> ! struct lineptr
> *this_line = &col_lineptrs[j+1][line_count];
> ! bool string_done =
> *(this_line->ptr + bytes_output[j+1]) == 0;
> !
> ! fputs(string_done ? "
> " : " ; ", fout);
> ! }
> }
I think it's a bad idea to use the same " : " separator in the two last
cases. They are different and they should be displayed differently.
--
Alvaro Herrera http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support
--
Sent via pgsql-patches mailing list (pgsql-patches@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-patches
