X Tutup
The Wayback Machine - https://web.archive.org/web/20250331145024/https://github.com/python/cpython/pull/131681
Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[WIP] gh-129813: Add PyBytesWriter C API (flavor with size) #131681

Draft
wants to merge 24 commits into
base: main
Choose a base branch
from
Draft

[WIP] gh-129813: Add PyBytesWriter C API (flavor with size) #131681

wants to merge 24 commits into from
+1,155 −411

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Mar 24, 2025 •
edited by bedevere-app bot
Loading

Add functions:

  • PyBytesWriter_Create()
  • PyBytesWriter_Discard()
  • PyBytesWriter_Finish()
  • PyBytesWriter_FinishWithSize()
  • PyBytesWriter_FinishWithEndPointer()
  • PyBytesWriter_Data()
  • PyBytesWriter_Allocated()
  • PyBytesWriter_SetSize()
  • PyBytesWriter_Resize()

@bedevere-app bedevere-app bot mentioned this pull request Mar 24, 2025
Closed
@vstinner vstinner force-pushed the bytes_writer_size branch from 459f3d8 to 9097e5f Compare March 24, 2025 16:55
@vstinner
pythongh-129813: Add PyBytesWriter C API (flavor with size)
e24d40e 
Add functions:

* PyBytesWriter_Create()
* PyBytesWriter_Discard()
* PyBytesWriter_Finish()
* PyBytesWriter_FinishWithSize()
* PyBytesWriter_FinishWithEndPointer()
* PyBytesWriter_Data()
* PyBytesWriter_Allocated()
* PyBytesWriter_SetSize()
* PyBytesWriter_Resize()
@vstinner vstinner force-pushed the bytes_writer_size branch from 9097e5f to e24d40e Compare March 24, 2025 16:56
@vstinner vstinner changed the title [WIP] gh-129813: Add PyBytesWriter C API (with size flavor) [WIP] gh-129813: Add PyBytesWriter C API (flavor with size) Mar 24, 2025
vstinner added 23 commits March 25, 2025 13:07
@vstinner
Remove PyBytesWriter_SetSize()
8761a9b
@vstinner
Add tests
92e1294 
* Add PyBytesWriter_GetSize()
* Rename:

  * PyBytesWriter_Data() => PyBytesWriter_GetData()
  * PyBytesWriter_Allocated() => PyBytesWriter_GetAllocated()
@vstinner
Add PyBytesWriter_WriteBytes()
eff71b5
@vstinner
Add PyBytesWriter_Format()
31c7ca7
@vstinner
Fix build on Windows
86d0fd9
@vstinner
Add PyBytesWriter_ResizeAndUpdatePointer() function
79fa5f8
@vstinner
Convert _PyBytes_FromIterator()
bf60f7f
@vstinner
Add _PyBytesWriter_CreateByteArray()
62a15be 
Convert _PyBytes_FromHex().
@vstinner
Convert _PyBytes_FormatEx()
0a70d70
@vstinner
Rename PyBytesWriter_FinishWithPointer()
457e21a
@vstinner
Add PyBytesWriter_GrowAndUpdatePointer()
40ef4e1
@vstinner
Make PyBytesWriter_ResizeAndUpdatePointer() private
0313087
@vstinner
Make PyBytesWriter_GetAllocated() private
c8ac889
@vstinner
Don't overallocate for bytearray()
7095ac4
@vstinner
Move _PyBytesWriter_CreateByteArray() to the internal C API
befd574
@vstinner
Move code
3ba1d1c
@vstinner
Add examples
ede2776
@vstinner
Add high-level API example
be56685
@vstinner
Fix tests
1135390
@vstinner
fix linter
000ba58
@vstinner
Convert more functions
b864c26 
Replace  PyBytes_FromStringAndSize(NULL, 0) with
Py_GetConstant(Py_CONSTANT_EMPTY_BYTES).
@vstinner
Convert _hashopenssl function
6d7e37d
@vstinner
Detect strlen() overflow
d8a4659
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gpshead gpshead Awaiting requested review from gpshead gpshead will be requested when the pull request is marked ready for review gpshead is a code owner

@picnixz picnixz Awaiting requested review from picnixz picnixz will be requested when the pull request is marked ready for review picnixz is a code owner

@ericsnowcurrently ericsnowcurrently Awaiting requested review from ericsnowcurrently ericsnowcurrently will be requested when the pull request is marked ready for review ericsnowcurrently is a code owner

@corona10 corona10 Awaiting requested review from corona10 corona10 will be requested when the pull request is marked ready for review corona10 is a code owner

@erlend-aasland erlend-aasland Awaiting requested review from erlend-aasland erlend-aasland will be requested when the pull request is marked ready for review erlend-aasland is a code owner

@serhiy-storchaka serhiy-storchaka Awaiting requested review from serhiy-storchaka serhiy-storchaka will be requested when the pull request is marked ready for review serhiy-storchaka is a code owner

Assignees
No one assigned
Labels
skip news
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@vstinner
X Tutup