bpo-47217: add name attribute to bz2 fileobject #32311
Conversation
| @@ -94,6 +94,11 @@ def __init__(self, filename, mode="r", *, compresslevel=9): | |||
| else: | |||
| self._pos = 0 | |||
|
|
|||
| if not isinstance(filename, (str, bytes)): | |||
| self.name = '' | |||
There was a problem hiding this comment.
Could this be some recognizable sentinel (<stream> or something better) rather than empty string?
There was a problem hiding this comment.
This seemed the easiest way of doing it without complicating the logic and the tests. Also, I couldn't find a uniform way of dealing with a lack of "name" for BytesIO() objects in other libraries, so I did not want to introduce anything "new and specific".
Do you have an example (or an idea) of a sentinel use that would fit here? In that case, we could do a similar change for gzip, bz2, lzma, ... libraries to have a bit more standardized way of dealing with this situation when we have BytesIO() objects.
|
Thank you for your PR @ellaellela, but this is a part of more complex issue #115961. It is better to solve it all in one place. |
https://bugs.python.org/issue47217