| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
| 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?
Sorry, something went wrong.
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.
Sorry, something went wrong.
There was a problem hiding this comment.
Ran 98 tests in 11.650s
OK
Looks ok.
Sorry, something went wrong.
|
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. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
https://bugs.python.org/issue47217