summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--ChangeLog15
-rw-r--r--bootstrap.conf1
-rw-r--r--configure.ac1
-rw-r--r--src/libs/libgroff/hypot.cpp4
4 files changed, 16 insertions, 5 deletions
diff --git a/ChangeLog b/ChangeLog
index fc192ecb4..1322f6ae0 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,18 @@
+2018-12-08 Bertrand Garrigues <bertrand.garrigues@laposte.net>
+
+ Use gnulib's 'hypot' module.
+
+ In 'configure.ac', the macro call
+ 'GROFF_NEED_DECLARATION([hypot])' checks whether the function
+ 'hypot' is correctly declared in 'math.h', but the test is buggy
+ and may yields false results.
+
+ * bootsrap.conf: add 'hypot' module.
+
+ * configure.ac: Remove GROFF_NEED_DECLARATION([hypot]).
+
+ * src/libs/libgroff/hypot.cpp: remove declaration of hypot.
+
2018-12-07 Deri James <deri@chuzzlewit.myzen.co.uk>
Problem running gropdf on big endian (Sparc)
diff --git a/bootstrap.conf b/bootstrap.conf
index 5161d1fd1..6cf16c840 100644
--- a/bootstrap.conf
+++ b/bootstrap.conf
@@ -43,6 +43,7 @@ gnulib_modules="
fprintf-posix
snprintf
vsnprintf
+ hypot
"
# Name of the Makefile.am
diff --git a/configure.ac b/configure.ac
index cb2bd10aa..e0a049f65 100644
--- a/configure.ac
+++ b/configure.ac
@@ -95,7 +95,6 @@ GROFF_POSIX
# checks for header stuff
GROFF_SRAND
GROFF_NEED_DECLARATION([gettimeofday])
-GROFF_NEED_DECLARATION([hypot])
GROFF_NEED_DECLARATION([popen])
GROFF_NEED_DECLARATION([pclose])
GROFF_NEED_DECLARATION([putenv])
diff --git a/src/libs/libgroff/hypot.cpp b/src/libs/libgroff/hypot.cpp
index 3eb2fb966..1fa44d466 100644
--- a/src/libs/libgroff/hypot.cpp
+++ b/src/libs/libgroff/hypot.cpp
@@ -20,10 +20,6 @@ along with this program. If not, see <http://www.gnu.org/licenses/>. */
#include <math.h>
-#ifdef NEED_DECLARATION_HYPOT
- double hypot(double, double);
-#endif /* NEED_DECLARATION_HYPOT */
-
double groff_hypot(double x, double y)
{
double result = hypot(x, y);