diff options
| author | Andrey Doroschenko <dorosch.github.io@yandex.ru> | 2020-10-20 14:05:01 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-10-20 14:05:01 (GMT) |
| commit | ec42789e6e14f6b6ac13569aeadc13798d7173a8 (patch) | |
| tree | 4354f760371a9d4d0430d8ec0b610baf4d3703c5 | |
| parent | faddc7449d07e883b8ad8bb95dd68ce6dbdc06e8 (diff) | |
| download | cpython-ec42789e6e14f6b6ac13569aeadc13798d7173a8.zip cpython-ec42789e6e14f6b6ac13569aeadc13798d7173a8.tar.gz cpython-ec42789e6e14f6b6ac13569aeadc13798d7173a8.tar.bz2 | |
bpo-39693: mention KeyError in tarfile extractfile documentation (GH-18639)
Co-authored-by: Andrey Darascheka <andrei.daraschenka@leverx.com>
| -rw-r--r-- | Doc/library/tarfile.rst | 9 | ||||
| -rwxr-xr-x | Lib/tarfile.py | 7 | ||||
| -rw-r--r-- | Misc/ACKS | 1 | ||||
| -rw-r--r-- | Misc/NEWS.d/next/Documentation/2020-02-24-09-02-05.bpo-39693.QXw0Fm.rst | 1 |
4 files changed, 11 insertions, 7 deletions
diff --git a/Doc/library/tarfile.rst b/Doc/library/tarfile.rst index cca466b..7a114fd 100644 --- a/Doc/library/tarfile.rst +++ b/Doc/library/tarfile.rst @@ -445,10 +445,11 @@ be finalized; only the internally used file object will be closed. See the .. method:: TarFile.extractfile(member) - Extract a member from the archive as a file object. *member* may be a filename - or a :class:`TarInfo` object. If *member* is a regular file or a link, an - :class:`io.BufferedReader` object is returned. Otherwise, :const:`None` is - returned. + Extract a member from the archive as a file object. *member* may be + a filename or a :class:`TarInfo` object. If *member* is a regular file or + a link, an :class:`io.BufferedReader` object is returned. For all other + existing members, :const:`None` is returned. If *member* does not appear + in the archive, :exc:`KeyError` is raised. .. versionchanged:: 3.3 Return an :class:`io.BufferedReader` object. diff --git a/Lib/tarfile.py b/Lib/tarfile.py index 1fae294..e422794 100755 --- a/Lib/tarfile.py +++ b/Lib/tarfile.py @@ -2093,9 +2093,10 @@ class TarFile(object): def extractfile(self, member): """Extract a member from the archive as a file object. `member' may be - a filename or a TarInfo object. If `member' is a regular file or a - link, an io.BufferedReader object is returned. Otherwise, None is - returned. + a filename or a TarInfo object. If `member' is a regular file or + a link, an io.BufferedReader object is returned. For all other + existing members, None is returned. If `member' does not appear + in the archive, KeyError is raised. """ self._check("r") @@ -1957,5 +1957,6 @@ Gennadiy Zlobin Doug Zongker Peter Åstrand Vlad Emelianov +Andrey Doroschenko (Entries should be added in rough alphabetical order by last names) diff --git a/Misc/NEWS.d/next/Documentation/2020-02-24-09-02-05.bpo-39693.QXw0Fm.rst b/Misc/NEWS.d/next/Documentation/2020-02-24-09-02-05.bpo-39693.QXw0Fm.rst new file mode 100644 index 0000000..86049c5 --- /dev/null +++ b/Misc/NEWS.d/next/Documentation/2020-02-24-09-02-05.bpo-39693.QXw0Fm.rst @@ -0,0 +1 @@ +Fix tarfile's extractfile documentation |