diff options
-rw-r--r-- | ChangeLog | 15 | ||||
-rw-r--r-- | bootstrap.conf | 1 | ||||
-rw-r--r-- | configure.ac | 1 | ||||
-rw-r--r-- | src/libs/libgroff/hypot.cpp | 4 |
4 files changed, 16 insertions, 5 deletions
@@ -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); |