aboutsummaryrefslogtreecommitdiff
path: root/src/tutorial/basics.source
diff options
context:
space:
mode:
authorDean Rasheed <dean.a.rasheed@gmail.com>2021-08-06 21:31:58 +0100
committerDean Rasheed <dean.a.rasheed@gmail.com>2021-08-06 21:31:58 +0100
commitcc4420f8829ddbc852e8c98781f09401ef26031d (patch)
tree26e6534385701717c82d3465c4164db8783da3d1 /src/tutorial/basics.source
parent0afe231a0db6442eed6c9d3fe22745405597943a (diff)
downloadpostgresql-cc4420f8829ddbc852e8c98781f09401ef26031d.tar.gz
postgresql-cc4420f8829ddbc852e8c98781f09401ef26031d.zip
Adjust the integer overflow tests in the numeric code.
Formerly, the numeric code tested whether an integer value of a larger type would fit in a smaller type by casting it to the smaller type and then testing if the reverse conversion produced the original value. That's perfectly fine, except that it caused a test failure on buildfarm animal castoroides, most likely due to a compiler bug. Instead, do these tests by comparing against PG_INT16/32_MIN/MAX. That matches existing code in other places, such as int84(), which is more widely tested, and so is less likely to go wrong. While at it, add regression tests covering the numeric-to-int8/4/2 conversions, and adjust the recently added tests to the style of 434ddfb79a (on the v11 branch) to make failures easier to diagnose. Per buildfarm via Tom Lane, reviewed by Tom Lane. Discussion: https://postgr.es/m/2394813.1628179479%40sss.pgh.pa.us
Diffstat (limited to 'src/tutorial/basics.source')
0 files changed, 0 insertions, 0 deletions