Project

General

Profile

Actions
Copy link

Bug #343

closed

ACX_FORTRAN_RUN_CHECK_SIZEOF in acx_fc_real_size.m4 does not check size of its argument

Added by Matthew Krupcale about 7 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Start date:
09/25/2017
Due date:
% Done:

100%

Estimated time:

Description

The function ACX_FORTRAN_RUN_CHECK_SIZEOF in m4/acx_fc_real_size.m4 does not actually check for the size of the real type passed as its argument, similar to the analogous function _ACX_FORTRAN_CHECK_SIZEOF_INTEGRAL_TYPE_RUN in m4/acx_fc_integer_size.m4 for integer types.

Furthermore, when using this function, the bits division arithmetic,

AS_VAR_SET([acx_fortran_Sizeof],[`expr ]AS_VAR_GET([acx_fortran_Sizeof])[ / $acx_cv_c_char_bits`])]

was returning some shell errors stating that expr was missing an operator, and that / is a directory. This seems to be due to the multiple nests of eval which come with AS_VAR_SET and AS_VAR_GET. It was easier to just do what was intended using the specifically designed method AS_VAR_ARITH, rather than using the more general AS_VAR_SET with expr.

The attached patch fixes these two issues.

Files

Download all files
scales-ppm-acx_fc_real_size.patch (893 Bytes) scales-ppm-acx_fc_real_size.patch Matthew Krupcale, 09/25/2017 06:29 PM
scales-ppm-1.0.5-acx_fc_real_size.patch (498 Bytes) scales-ppm-1.0.5-acx_fc_real_size.patch Matthew Krupcale, 10/14/2017 05:25 PM
Actions
Copy link

Also available in: Atom PDF