-
-
Notifications
You must be signed in to change notification settings - Fork 32.1k
gh-102471, PEP 757: Add PyLong import and export API #121339
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
Changes from 1 commit
f4fdbf2
c2e568e
f0d9525
b19764f
6f7fd11
080e079
1a7902f
b70a6dd
07552a7
0d0f942
762c33a
20be7a3
d92bf1e
b3b02a2
caca2d7
4221a49
37b1d49
d70a121
4aa25f6
90973d4
5d3e224
c7d7cb2
a3d601a
c049268
06b196b
3e8d296
86c68c2
a8fd669
a04f9d0
b2be94a
ca98ad1
167d75e
5e53a5b
c24789f
0422f9d
a529a48
3db44f3
1d2863e
d663511
816798d
033bd65
36b87d4
94d852e
a72ff83
53d584b
577598a
b08cd55
eaebef3
03248c7
0a26f97
88a62fe
45517ab
92007d1
6d3cb80
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
Remove doc from the header file, only keep doc in Doc/c-api/long.rst to avoid the documentation to be outdated in one place.
- Loading branch information
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -724,7 +724,7 @@ Export API | |||||||||||||||
|
||||||||||||||||
.. c:member:: uint8_t negative | ||||||||||||||||
|
||||||||||||||||
1 if the number is negative, 0 otherwise. | ||||||||||||||||
``1`` if the number is negative, ``0`` otherwise. | ||||||||||||||||
Only valid if :c:member:`digits` is not ``NULL``. | ||||||||||||||||
|
||||||||||||||||
.. c:member:: Py_ssize_t ndigits | ||||||||||||||||
|
@@ -741,11 +741,11 @@ Export API | |||||||||||||||
|
||||||||||||||||
Export a Python :class:`int` object. | ||||||||||||||||
|
||||||||||||||||
On success, set *\*export_long* and return 0. | ||||||||||||||||
On error, set an exception and return -1. | ||||||||||||||||
On success, set *\*export_long* and return ``0``. | ||||||||||||||||
vstinner marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||||||||||||
On error, set an exception and return ``-1``. | ||||||||||||||||
|
||||||||||||||||
If *export_long.digits* is not ``NULL``, :c:func:`PyLong_FreeExport` must be | ||||||||||||||||
called when the export is no longer needed. | ||||||||||||||||
If *export_long->digits* is not ``NULL``, :c:func:`PyLong_FreeExport` must | ||||||||||||||||
be called when the export is no longer needed. | ||||||||||||||||
vstinner marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||||||||||||
|
||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
Lets see if we can restore this in a that way. It might be helpful for e.g. Sage, which doesn't support PyPy. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I would prefer to not add this note. It was controversial during PEP 757 design. @serhiy-storchaka @encukou: What do you think? Would you be ok to declare that the PyLong_Export() function cannot fail if the argument is a Python int? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
It was proposed unconditionally, not as CPython's implementation detail. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe 10000 this comment to others. Learn more. I'm fine with it, as an implementation detail. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||||||||||||||||
|
||||||||||||||||
.. c:function:: void PyLong_FreeExport(PyLongExport *export_long) | ||||||||||||||||
|
@@ -794,6 +794,9 @@ The :c:type:`PyLongWriter` API can be used to import an integer. | |||||||||||||||
On success, return a Python :class:`int` object. | ||||||||||||||||
On error, set an exception and return ``NULL``. | ||||||||||||||||
|
||||||||||||||||
The function takes care of normalizing the digits and converts the object | ||||||||||||||||
to a compact integer if needed. | ||||||||||||||||
|
||||||||||||||||
|
||||||||||||||||
.. c:function:: void PyLongWriter_Discard(PyLongWriter *writer) | ||||||||||||||||
|
||||||||||||||||
|
Uh oh!
There was an error while loading. Please reload this page.