bgarrigues pushed a commit to branch master
in repository groff.
commit dd3d602cd25447fa8f4b5471a0ade882dc89032a
Author: Bertrand Garrigues <[email protected]>
Date: Sun Dec 9 00:06:31 2018 +0100
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.
---
ChangeLog | 15 +++++++++++++++
bootstrap.conf | 1 +
configure.ac | 1 -
src/libs/libgroff/hypot.cpp | 4 ----
4 files changed, 16 insertions(+), 5 deletions(-)
diff --git a/ChangeLog b/ChangeLog
index fc192ec..1322f6a 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,18 @@
+2018-12-08 Bertrand Garrigues <[email protected]>
+
+ 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 <[email protected]>
Problem running gropdf on big endian (Sparc)
diff --git a/bootstrap.conf b/bootstrap.conf
index 5161d1f..6cf16c8 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 cb2bd10..e0a049f 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 3eb2fb9..1fa44d4 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);
_______________________________________________
Groff-commit mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/groff-commit
