This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: fileinput openfile patch, bz2fileinput
Type: Stage:
Components: Library (Lib) Versions: Python 2.3
process
Status: closed Resolution: rejected
Dependencies: Superseder:
Assigned To: Nosy List: georg.brandl, hoffman, sib4
Priority: normal Keywords: patch

Created on 2005-06-22 11:50 by sib4, last changed 2022-04-11 14:56 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
bz2fileinput.patch sib4, 2005-06-22 11:50 bz2fileinput patch
Messages (4)
msg48495 - (view) Author: Stephan Boettcher (sib4) Date: 2005-06-22 11:50
Please consider the enclosed patch to fileinput.py,
which  implements an openfile method of FileInput. 
Included is a use case bz2fileinput.py.
msg48496 - (view) Author: Michael Hoffman (hoffman) Date: 2005-06-30 07:51
Logged In: YES 
user_id=987664

Hmmm. This could be useful. Pythonistas could also supply
openfile for, say, URLs as well. I'm kind of lukewarm on it
for some reason--I think that you'd be better off
decompressing the data somewhere else. I also did some
benchmarking the other day and found that fileinput is
really, really slow.

I don't understand the change from Fileinput to _Fileinput.
You're going to need to fix the inplace stuff in
Fileinput.readline().

This needs documentation, and a more detailed description.

Please provide a patch against the latest CVS instead of
Python 2.3.
msg48497 - (view) Author: Georg Brandl (georg.brandl) * (Python committer) Date: 2005-08-31 22:36
Logged In: YES 
user_id=1188172

In addition to hoffmanm's question about the renaming of
FileInput, two issues:
- An additional module, especially with only a few lines in
it, is not going to be accepted into the stdlib. This is
better done in your own code, where you know what types of
files you will encounter.
See patch #1215184, which implements a similar openhook for
fileinput.
- Only bz2 is too selective. You would have to have modules
"gzfileinput", "urlfileinput", etc. Too much name crowding.

For these reasons, rejecting this patch.
msg48498 - (view) Author: Stephan Boettcher (sib4) Date: 2005-09-01 14:01
Logged In: YES 
user_id=252223

Sorry for not responding in time to those questions.

My proposal was meant to provide the openfile method, and
provide the user with a way to override it and still use the
static infrastructure if the fileinput module, as
demonstrated by bz2fileinput.py.

Instead of using the hook "_FileInput" to provide a
replacement class to the module, it may be better to replace
FileInput.openfile() in derived modules, but I thought that
may be even more of a hack.

If you think this is worth following up, please advise of
the best way to achieve this goal.

Thanks
Stephan
History
Date User Action Args
2022-04-11 14:56:11adminsetgithub: 42109
2005-06-22 11:50:15sib4create