Skip to content

bpo-29176: curses: use tmpfile() for getwin/putwin #237

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

Closed
wants to merge 1 commit into from
Closed

bpo-29176: curses: use tmpfile() for getwin/putwin #237

wants to merge 1 commit into from

Conversation

vstinner
Copy link
Member

Use C function tmpfile() for curses.getwin() and curses.putwin(),
instead of mkstemp() using an hardcoded /tmp path. The tmpfile()
chooses the directory for the temporary file and so is more portable
(especially, it works on Android).

Replace also IOError with OSError: since the PEP 3151, IOError is an
alias to OSError.

http://bugs.python.org/issue29176

Use C function tmpfile() for curses.getwin() and curses.putwin(),
instead of mkstemp() using an hardcoded /tmp path. The tmpfile()
chooses the directory for the temporary file and so is more portable
(especially, it works on Android).

Replace also IOError with OSError: since the PEP 3151, IOError is an
alias to OSError.
@vstinner
Copy link
Member Author

Oh, Christian Heimes (@tiran) created the same PR than me: PR #235 ! So I abandon mine.

@vstinner vstinner closed this Feb 22, 2017
@vstinner vstinner deleted the curses_tmpfile branch February 22, 2017 11:26
akruis pushed a commit to akruis/cpython that referenced this pull request Apr 20, 2021
The 4th argument has been changed from immediate=True to pending=False
a long time ago.
akruis pushed a commit to akruis/cpython that referenced this pull request May 27, 2021
The 4th argument has been changed from immediate=True to pending=False
a long time ago.

(cherry picked from commit 12b78b0)
ahgamut pushed a commit to ahgamut/cpython that referenced this pull request Aug 25, 2021
The 4th argument has been changed from immediate=True to pending=False
a long time ago.

(cherry picked from commit 12b78b0)
jaraco pushed a commit that referenced this pull request Dec 2, 2022
If celery or redis is down, then we can't perform "sleep".
Heroku's web dyno times out after 30 seconds. 😥
jaraco added a commit to jaraco/cpython that referenced this pull request Feb 17, 2023
…ersable-docs

abc: avoid abbreviations in the Traversable documentation
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.

3 participants