Robert Haas <robertmh...@gmail.com> writes:
> On Mon, Jul 21, 2014 at 10:52 AM, Emre Hasegeli <e...@hasegeli.com> wrote:
>> The first two shapes on src/test/regress/sql/polygon.sql do not make
>> sense to me.

> Well, I think the number of tabs that makes them look best depends on
> your tab-stop setting.  At present, I find that with 8-space tabs
> things seem to line up pretty well, whereas with your patch, 4-space
> tabs line up well.

Perhaps we should expand tabs to spaces in those ascii-art diagrams?

> Personally, though, my vote would be to just leave it the way it is.
> I don't think there's really a problem here in need of solving.

I concur with doubting that changing the actual regression test cases
is a good thing to do, at least not without more analysis.  But "the
pictures make no sense with the wrong tab setting" is a pretty simple
issue, and one that I can see biting people.

                        regards, tom lane


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to