search

GraphBLAS / LAGraph

This is a library plus a test harness for collecting algorithms that use the GraphBLAS. For test coverage reports, see https://graphblas.org/LAGraph/ . Documentation: https://lagraph.readthedocs.org
Other
229 stars 61 forks source link

Change complex to _Complex for C++ compatibility #77

Closed szarnyasg closed 4 years ago

szarnyasg commented 4 years ago

Fixes #74. Suggested by the answer at https://stackoverflow.com/a/10540302/3580502.

DrTimothyAldenDavis commented 4 years ago

In ANSI C11, the convention using _Complex is to reverse the order, as in double _Complex or float _Complex, not _Complex double

Does it work in C++ to use double _Complex and float _Complex?

DrTimothyAldenDavis commented 4 years ago

See for example, in the ANSI C11 spec:

image

szarnyasg commented 4 years ago

Does it work in C++ to use double _Complex and float _Complex?

Yes, I made & pushed the changes. It works fine with our C++ code so I think this PR is good to go.