Don't call PG_RETURN_BOOL() in a function not returning Datum.

This code is new in v12, and the defect probably was not user-visible.

diff --git a/src/backend/utils/adt/timestamp.c b/src/backend/utils/adt/timestamp.c
index e5ac371fa0f3c6c280d311cc8438355bea32601e..853a8dc49a469e2309dbbd7f32a5c1daa6a37db6 100644 (file)
--- a/src/backend/utils/adt/timestamp.c
+++ b/src/backend/utils/adt/timestamp.c
@@ -5180,8 +5180,8 @@ TimestampTimestampTzRequiresRewrite(void)
    long        offset;
 
    if (pg_get_timezone_offset(session_timezone, &offset) && offset == 0)
-       PG_RETURN_BOOL(false);
-   PG_RETURN_BOOL(true);
+       return false;
+   return true;
 }
 
 /* timestamp_timestamptz()