From: Tom Lane Date: Wed, 4 Mar 2009 22:08:40 +0000 (+0000) Subject: Put back our old workaround for machines that declare cbrt() in math.h but X-Git-Tag: REL8_1_17~4 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=9a8644a68281a2a24e318b2660d3968f7ee55fbc;p=postgresql Put back our old workaround for machines that declare cbrt() in math.h but fail to provide the function itself. Not sure how we escaped testing anything later than 7.3 on such cases, but they still exist, as per André Volpato's report about AIX 5.3. --- diff --git a/src/backend/utils/adt/float.c b/src/backend/utils/adt/float.c index d80b3d4ef4..a5373da610 100644 --- a/src/backend/utils/adt/float.c +++ b/src/backend/utils/adt/float.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/utils/adt/float.c,v 1.115.2.1 2006/04/24 20:36:41 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/utils/adt/float.c,v 1.115.2.2 2009/03/04 22:08:40 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -110,6 +110,15 @@ static int float4_cmp_internal(float4 a, float4 b); static int float8_cmp_internal(float8 a, float8 b); #ifndef HAVE_CBRT +/* + * Some machines (in particular, some versions of AIX) have an extern + * declaration for cbrt() in but fail to provide the actual + * function, which causes configure to not set HAVE_CBRT. Furthermore, + * their compilers spit up at the mismatch between extern declaration + * and static definition. We work around that here by the expedient + * of a #define to make the actual name of the static function different. + */ +#define cbrt my_cbrt static double cbrt(double x); #endif /* HAVE_CBRT */