Skip to content

[3.9] bpo-33239: Fix default value of 'buffering' parameter in docs of tempfile.* functions (GH-21763)#22225

Merged
vsajip merged 1 commit intopython:3.9from
miss-islington:backport-b48389d-3.9
Sep 13, 2020
Merged

[3.9] bpo-33239: Fix default value of 'buffering' parameter in docs of tempfile.* functions (GH-21763)#22225
vsajip merged 1 commit intopython:3.9from
miss-islington:backport-b48389d-3.9

Conversation

@miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Sep 13, 2020

None doesn't work:

>>> import tempfile
>>> tempfile.TemporaryFile(buffering=None)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/sergey/tmp/cpython-dev/Lib/tempfile.py", line 607, in TemporaryFile
    return _io.open(fd, mode, buffering=buffering,
TypeError: 'NoneType' object cannot be interpreted as an integer

Automerge-Triggered-By: @vsajip
(cherry picked from commit b48389d)

Co-authored-by: Sergey Fedoseev fedoseev.sergey@gmail.com

https://bugs.python.org/issue33239

…file.* functions (pythonGH-21763)

`None` doesn't work:

```python
>>> import tempfile
>>> tempfile.TemporaryFile(buffering=None)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/sergey/tmp/cpython-dev/Lib/tempfile.py", line 607, in TemporaryFile
    return _io.open(fd, mode, buffering=buffering,
TypeError: 'NoneType' object cannot be interpreted as an integer
```

Automerge-Triggered-By: @vsajip
(cherry picked from commit b48389d)

Co-authored-by: Sergey Fedoseev <fedoseev.sergey@gmail.com>
@miss-islington
Copy link
Contributor Author

@sir-sigurd: Status check is done, and it's a success ✅ .

@miss-islington
Copy link
Contributor Author

@sir-sigurd: Status check is done, and it's a success ✅ .

@vsajip vsajip merged commit c978be2 into python:3.9 Sep 13, 2020
@miss-islington miss-islington deleted the backport-b48389d-3.9 branch September 13, 2020 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation in the Doc dir skip news

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants