pgsql: Don't use is_infinite() where isinf() will do. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Don't use is_infinite() where isinf() will do.
Date
Msg-id E1g8qE2-0007W5-8e@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Don't use is_infinite() where isinf() will do.

Places that aren't testing for sign should not use the more expensive
function; it's just wasteful, not to mention being a cognitive load
for readers who may know what isinf() is but not is_infinite().

As things stand, we actually don't need is_infinite() anyplace except
float4out/float8out, which means it could potentially go away altogether
after the changes I proposed in <13178.1538794717@sss.pgh.pa.us>.

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/0209f0285d9b1c60bf74cc9f5f0133d7bdd887c3

Modified Files
--------------
src/backend/utils/adt/formatting.c          |  4 ++--
src/backend/utils/adt/rangetypes_selfuncs.c | 14 ++++++++------
2 files changed, 10 insertions(+), 8 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Propagate xactStartTimestamp and stmtStartTimestamp toparallel
Next
From: Tom Lane
Date:
Subject: pgsql: Remove more redundant relation locking during executor startup.