-
-
Notifications
You must be signed in to change notification settings - Fork 32k
gh-61103: support double complex (_Complex) type in ctypes #120894
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 1 commit
Commits
Show all changes
57 commits
Select commit
Hold shift + click to select a range
c9b872f
gh-61103: support double complex (_Complex) type in ctypes
skirpichev b402300
+ ifdef
skirpichev ad39e6d
+ fix rst formatting
skirpichev fd9135b
+ CMPLX
skirpichev c296fb3
Apply suggestions from code review
skirpichev 7453ca5
+ support _Complex in c-analyzer
skirpichev e444c9c
+1
skirpichev 2e90643
expand docs
skirpichev 32ffacf
drop ctypes.__STDC_IEC_559_COMPLEX__
skirpichev 9163950
And more tests, finally)
skirpichev bf81682
Apply suggestions from code review
skirpichev af2fdce
Apply suggestions from code review
skirpichev 6d9d5ab
+ allow _Complex specifier before float/double/etc
skirpichev f9c709c
+ comment on clang workaround
skirpichev ccd418d
Address review:
skirpichev eb313dc
+ @requires_IEEE_754 for test_csqrt()
skirpichev 693c04e
Revert "+ @requires_IEEE_754 for test_csqrt()"
skirpichev 0ee7049
Adjust test_csqrt() to not use special numbers
skirpichev de91bbe
Revert "Adjust test_csqrt() to not use special numbers"
skirpichev b79200e
+1
skirpichev be6a685
+1
skirpichev 04e89c2
revert test + fix typo
skirpichev 0bc87da
Adjust test_csqrt() to not use special numbers
skirpichev 6347412
+ make CMPLX static
skirpichev e757fb6
try bad CMPLX()
skirpichev 1eb989c
more tests
skirpichev b44aae2
cexp test
skirpichev 2153608
use my_csquare() in test
skirpichev af42aa8
revert CMPLX
skirpichev e57198e
Address review:
skirpichev 36732bc
revert tests
skirpichev 892b241
XXX skip test on MacOs arm64
skirpichev ee953c3
Revert "XXX skip test on MacOs arm64"
skirpichev 875cbb8
XXX (to pass)
skirpichev f8e3a47
Address review: guard _complex.h with ifndef
skirpichev f447919
Apply suggestions from code review
skirpichev 7b1c366
Merge branch 'master' into ctypes-c_complex-61103
skirpichev 2ea028a
Add _Complex (sic!) to union result
skirpichev 837add4
add missing ifdef + restore tests
skirpichev 1b8f37e
Adjust test_csqrt() to not use special numbers
skirpichev 0bd1ebe
Add _Complex also to tagPyCArgObject struct
skirpichev 6eba445
cleanup: use "complex" instead of _Complex
skirpichev a6ac463
address review:
skirpichev 615e8de
Update Modules/_ctypes/_ctypes.c
skirpichev f7d9973
address review: renamed macro (->PY_HAVE_C_COMPLEX)
skirpichev 3d170bd
+ autoreconf
skirpichev 4dd044c
+ test to ensure CMPLX preserves special components
skirpichev 4cf7045
address review: rename macros to Py_HAVE_C_COMPLEX
skirpichev 264fa7d
fix typos and adjust Makefile.pre.in
skirpichev f54b964
enable Py_HAVE_C_COMPLEX only if libff support complex numbers
skirpichev 23db200
Revert "enable Py_HAVE_C_COMPLEX only if libff support complex numbers"
skirpichev 5a7e366
ifdef Py_HAVE_C_COMPLEX -> if defined(Py_HAVE_C_COMPLEX) && defined(F…
skirpichev 388e2ca
+ missing ffi.h imports
skirpichev e888565
+ add libffi's cflags/ldflags to _ctypes_test.c deps
skirpichev 56b524c
Merge branch 'master' into ctypes-c_complex-61103
skirpichev e9cab4a
Adjust test in _ctypes.c to not use FFI_TYPE_COMPLEX define
skirpichev 6d5bf66
sed: FFI_TYPE_COMPLEX -> FFI_TARGET_HAS_COMPLEX_TYPE
skirpichev File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Apply suggestions from code review
Co-authored-by: Nice Zombies <nineteendo19d0@gmail.com>
- Loading branch information
commit bf816828837c3fe6f21b8b75c3504081f11c68fe
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.