Skip to content

gh-95380: Remove the 1024 bytes limit in fcntl.fcntl() and fcntl.ioctl()#132907

Merged
serhiy-storchaka merged 7 commits into
python:mainfrom
serhiy-storchaka:fcntl-size-limit
May 13, 2025
Merged

gh-95380: Remove the 1024 bytes limit in fcntl.fcntl() and fcntl.ioctl()#132907
serhiy-storchaka merged 7 commits into
python:mainfrom
serhiy-storchaka:fcntl-size-limit

Conversation

@serhiy-storchaka

@serhiy-storchaka serhiy-storchaka commented Apr 25, 2025

Copy link
Copy Markdown
Member

Comment thread Modules/fcntlmodule.c
Comment thread Modules/fcntlmodule.c
Comment thread Doc/library/fcntl.rst Outdated
Comment thread Doc/library/fcntl.rst Outdated
Comment thread Misc/NEWS.d/next/Library/2025-04-25-11-53-37.gh-issue-95380.7dvPe-.rst Outdated
Co-authored-by: Victor Stinner <vstinner@python.org>
Comment thread Modules/fcntlmodule.c
Comment thread Doc/library/fcntl.rst

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. There is just a small issue with the doc.

Comment thread Doc/library/fcntl.rst
@serhiy-storchaka serhiy-storchaka enabled auto-merge (squash) May 13, 2025 14:18
@serhiy-storchaka

Copy link
Copy Markdown
Member Author

Thank you for your review, @vstinner.

@serhiy-storchaka serhiy-storchaka merged commit 18bf8f8 into python:main May 13, 2025
39 checks passed
Pranjal095 pushed a commit to Pranjal095/cpython that referenced this pull request Jul 12, 2025
taegyunkim pushed a commit to taegyunkim/cpython that referenced this pull request Aug 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants