even further. And what confidence do you have that this change eliminates all forms of the problem, anyway?
Yes, I think. Because that part of code ( if (IS_BADRATIO) {...} ) is a corner case itself. In example from Andrew, all boxes are placed to one page because of floating-point rounding.
We could check IS_BADRATIO again and if it's just put one half of all boxes on one page and another half to the another page as it does if all boxes are equal. But FPeq() seemed to me a simpler solution and FP* comparisons are widely used in geometry.
-- Teodor Sigaev E-mail: teo...@sigaev.ru WWW: http://www.sigaev.ru/ -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers