Hi, This is my review about 3th version of the patch. It is an useful improvement in my opinion. It worked well on my environment.
2013-12-11 17:43:06, Sergey Muraviov <sergey.k.murav...@gmail.com>: > It works in expanded mode when either format option is set to wrapped > (\pset format wrapped), or we have no pager, or pager doesn't chop long > lines (so you can still use the trick). I do not like this logic on the IsWrappingNeeded function. It does not seems right to check the environment variables for less. It would be hard to explain this behavior to the users. It is better to make this only the behavior of the wrapped format in expanded mode, in my opinion. > { > if (opt_border < 2) > fprintf(fout, "%s\n", > dlineptr[line_count].ptr); > else > fprintf(fout, "%-s%*s %s\n", > dlineptr[line_count].ptr, > dwidth - > dlineptr[line_count].width, "", > > dformat->rightvrule); > } Is it necessary to keep this old print line code? It seems to me the new code works well on (dlineptr[line_count].width <= dwidth) condition. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers