Commit bd2cdff
Fix NaN comparison in circle_same test
Commit c4c3400 changed geometric operators to use float4 and float8
functions, and handle NaN's in a better way. The circle sameness test
had a typo in the code which resulted in all comparisons with the left
circle having a NaN radius considered same.
postgres=# select '<(0,0),NaN>'::circle ~= '<(0,0),1>'::circle;
?column?
----------
t
(1 row)
This fixes the sameness test to consider the radius of both the left
and right circle.
Backpatch to v12 where this was introduced.
Author: Ranier Vilela <ranier.vf@gmail.com>
Discussion: https://postgr.es/m/CAEudQAo8dK=yctg2ZzjJuzV4zgOPBxRU5+Kb+yatFiddtQk6Rw@mail.gmail.com
Backpatch-through: v121 parent f3d572d commit bd2cdff
2 files changed
Lines changed: 2 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4763 | 4763 | | |
4764 | 4764 | | |
4765 | 4765 | | |
4766 | | - | |
| 4766 | + | |
4767 | 4767 | | |
4768 | 4768 | | |
4769 | 4769 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4344 | 4344 | | |
4345 | 4345 | | |
4346 | 4346 | | |
4347 | | - | |
4348 | 4347 | | |
4349 | | - | |
| 4348 | + | |
4350 | 4349 | | |
4351 | 4350 | | |
4352 | 4351 | | |
| |||
0 commit comments