'complex' type declaration should be changed to 'double complex'

[jhoblitt]

Under C99 'complex' is short hand for 'float complex' but the SDRS assumes that
'complex' means 'double complex'. This has been fixed in the SDRS and I'll be
sending out a new API delta shortly.

[robert.desonia@…]

Actually, under C99, 'complex' is short hand for 'double _Complex'. If you want proof, try the following
program:

#include<stdio.h>
#include<complex.h>

int main(void) {

printf("sizeof(complex)=%i\n",sizeof(complex));
printf("sizeof(float complex)=%i\n",sizeof(float complex));
printf("sizeof(double complex)=%i\n",sizeof(double complex));

return 0;

}

[jhoblitt]

Have you tried that with a complier other then gcc? In fact, if you enable
'-pedantic' with gcc it will tell you it is doing the wrong thing under ISO C by
defining 'complex' to mean 'double complex'.

If you can you point to the section of the c99 spec that defines 'complex' as
'double _Complex' you can remark this bug as invalid. Otherwise it is a gcc
specific behavior and a bug that needs to be fixed.

[robert.desonia@…]

The C99 specification seems to be fairly silent on what the correct behavior is here. I also know that GCC
has serious problems in places if we just replace 'complex' with 'complex double', strangely enough, so it
is not trivial to do what you ask either.

Well look into this further.

-rdd

[Paul Price]

​http://gcc.gnu.org/onlinedocs/gcc-3.2.3/gcc/Complex.html

[David.Robbins@…]

I guess Robert fixed this.

[jhoblitt]

This has not been fixed in head.

$ grep -R "double complex" src | wc -l
0

[David.Robbins@…]

I get 27.
I apologize ahead of time if I'm confused, but I believe this has been done.

[Paul Price]

I concur.

price@mithrandir:/home/mithrandir/price/psLib/psLib_head>grep -R "double
complex" src | wc -l

27

[jhoblitt]

Looks like I was caught by Gene's CVS tree reorganization.

[Paul Price]

Bug has been resolved.... closing.

[Paul Price]

Bugs have been fixed... closing.