| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
…struct module Each complex type interpreted as an array type containing exactly two elements of the corresponding real type (real and imaginary parts, respectively).
Co-authored-by: Lisandro Dalcin <dalcinl@gmail.com>
There was a problem hiding this comment.
I forgot but what was the decision about ctypes?
Sorry, something went wrong.
Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
|
Buildbots didn't get launched apparently but that's good as we're modifying the code. Let's wait until everytihng is stable and we'll launch the buildbots again |
Sorry, something went wrong.
@picnixz, are you about this suggestion: #121249 (comment)? Patch is here: #121249 (comment). I think it's much less urgent, as for the ctypes module - support can't be available unconditionally. PS: Not sure why bots aren't started. Perhaps, this label should be added by core dev. |
Sorry, something went wrong.
Ok! I actually wondered about the fate of ctypes because I wondered whether we still needed the _complex.h header.
I don't remember so? but maybe it is.. anyway, I'll add the label once you've confirmed whether the previous blurb needs an update. |
Sorry, something went wrong.
Feel free to do this even right now. But you should be able to prove that my concern is invalid. |
Sorry, something went wrong.
The thing is, I'm still not sure what's your concern, I don't understand what exactly you meant by "annex G support still required from system". I actually agree with such claim, as long a "system" mean the C compiler used to build FFI, and the C compiler used to build a shared lib that ctypes call into, but NOT the C compiler used to build CPython and ctypes. |
Sorry, something went wrong.
I tried to explain it twice: #121249 (comment) and #121249 (comment). Sorry, it's my best.
The only way to pass it on ctypes side will be a two-element array then. I'm not sure if this will work, as it's not how complexes passed to a function. |
Sorry, something went wrong.
There was a problem hiding this comment.
@hugovk Was it correct to amend the Misc/NEWS.d/3.14.0a1.rst for the main branch as the feature that we added there was actually changed? (that way, the online docs will contain only the most recent information and not duplicated entries). The entry will still exist in the tagged release though and will correctly reflect what 3.14.0a1 does.
EDIT: no it doesn't work that way :')
Sorry, something went wrong.
|
@skirpichev I'm so sorry... but I didn't see that the entire changelog contains all previous alphas as well.. so, if we were to remove the NEWS we'll have a missing entry in the changelog page... So just restore the 3.14.0a1.rst file to what it has been but keep the new one for the beta section. You can say that previously it was conditioned though. |
Sorry, something went wrong.
|
🤖 New build scheduled with the buildbot fleet by @skirpichev for commit 38fa2a7 🤖 Results will be shown at: https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F132864%2Fmerge If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again. |
Sorry, something went wrong.
|
I see no failures, related to the pr. |
Sorry, something went wrong.
…ythonGH-132864) Co-authored-by: Lisandro Dalcin <dalcinl@gmail.com> Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
| Back | FazBrowse Home | New Git URL |
Each complex type interpreted as an array type containing exactly two elements of the corresponding real type (real and imaginary parts, respectively).
📚 Documentation preview 📚: https://cpython-previews--132864.org.readthedocs.build/