forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit452b637
committed
Adjust new test case for more portability.
Early returns from the buildfarm say that most critters are good withcommitcbdb8b4, but gaur gives unexpected results with the test caseinvolving a float8 that's one-ULP-less-than-2^63. It appears that thatplatform's version of rint() rounds that value up to 2^63 instead ofleaving it be. This is possibly a bug, and it's also possible that noother platform anybody is using anywhere behaves likewise. Still, thepoint of the test is not to insist that everybody's rint() behaves exactlythe same. Let's use two-ULPs-less-than-2^63 instead, which I've testedto act the same on gaur as on more modern hardware.(This is, more or less, exactly the portability issue I'd feared mightarise...)Discussion:https://postgr.es/m/15519-4fc785b483201ff1@postgresql.org1 parentcbdb8b4 commit452b637
File tree
3 files changed
+5
-5
lines changed- src/test/regress
- expected
- sql
3 files changed
+5
-5
lines changedLines changed: 2 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
511 | 511 |
| |
512 | 512 |
| |
513 | 513 |
| |
514 |
| - | |
| 514 | + | |
515 | 515 |
| |
516 | 516 |
| |
517 |
| - | |
| 517 | + | |
518 | 518 |
| |
519 | 519 |
| |
520 | 520 |
| |
|
Lines changed: 2 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
513 | 513 |
| |
514 | 514 |
| |
515 | 515 |
| |
516 |
| - | |
| 516 | + | |
517 | 517 |
| |
518 | 518 |
| |
519 |
| - | |
| 519 | + | |
520 | 520 |
| |
521 | 521 |
| |
522 | 522 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
183 | 183 |
| |
184 | 184 |
| |
185 | 185 |
| |
186 |
| - | |
| 186 | + | |
187 | 187 |
| |
188 | 188 |
| |
189 | 189 |
| |
|
0 commit comments
Comments
(0)