Fix bogus return macros in range_overright_internal().
Fix bogus return macros in range_overright_internal().

PG_RETURN_BOOL() should only be used in functions following the V1 SQL
function API.  This coding accidentally fails to fail since letting the
compiler coerce the Datum representation of bool back to plain bool
does give the right answer; but that doesn't make it a good idea.

Back-patch to older branches just to avoid unnecessary code divergence.

adam8157/gpdb,jmcatamney/gpdb,50wu/gpdb,greenplum-db/gpdb,zeroae/postgres-xl,lisakowen/gpdb,xinzweb/gpdb,jmcatamney/gpdb,oberstet/postgres-xl,pavanvd/postgres-xl,yazun/postgres-xl,jmcatamney/gpdb,greenplum-db/gpdb,zeroae/postgres-xl,ovr/postgres-xl,50wu/gpdb,adam8157/gpdb,yazun/postgres-xl,ovr/postgres-xl,lisakowen/gpdb,adam8157/gpdb,oberstet/postgres-xl,lisakowen/gpdb,xinzweb/gpdb,lisakowen/gpdb,techdragon/Postgres-XL,Postgres-XL/Postgres-XL,pavanvd/postgres-xl,techdragon/Postgres-XL,50wu/gpdb,adam8157/gpdb,50wu/gpdb,jmcatamney/gpdb,Postgres-XL/Postgres-XL,50wu/gpdb,zeroae/postgres-xl,techdragon/Postgres-XL,zeroae/postgres-xl,ashwinstar/gpdb,xinzweb/gpdb,ashwinstar/gpdb,oberstet/postgres-xl,techdragon/Postgres-XL,lisakowen/gpdb,greenplum-db/gpdb,ovr/postgres-xl,ashwinstar/gpdb,ashwinstar/gpdb,50wu/gpdb,ovr/postgres-xl,xinzweb/gpdb,pavanvd/postgres-xl,yazun/postgres-xl,ashwinstar/gpdb,zeroae/postgres-xl,ovr/postgres-xl,ashwinstar/gpdb,techdragon/Postgres-XL,adam8157/gpdb,adam8157/gpdb,adam8157/gpdb,oberstet/postgres-xl,ashwinstar/gpdb,xinzweb/gpdb,50wu/gpdb,Postgres-XL/Postgres-XL,lisakowen/gpdb,greenplum-db/gpdb,Postgres-XL/Postgres-XL,greenplum-db/gpdb,yazun/postgres-xl,greenplum-db/gpdb,jmcatamney/gpdb,xinzweb/gpdb,jmcatamney/gpdb,Postgres-XL/Postgres-XL,pavanvd/postgres-xl,ashwinstar/gpdb,pavanvd/postgres-xl,greenplum-db/gpdb,yazun/postgres-xl,greenplum-db/gpdb,50wu/gpdb,xinzweb/gpdb,jmcatamney/gpdb,lisakowen/gpdb,adam8157/gpdb,oberstet/postgres-xl,jmcatamney/gpdb,lisakowen/gpdb,xinzweb/gpdb