| msg70578 - (view) |
Author: Anand B Pillai (pythonhacker) * |
Date: 2008-08-01 17:56 |
| Python has great in-built support for all sorts of text compression including bzip, gzip, tarfile and other general purpose modules like zlib etc. Of these, I have a gripe with gzip - it does not provide a simple way of compressing/uncompressing a string like the other modules to (namely bzip, zlib) at the module level. It is relatively easy to perform gzip de-compression using the GzipFile class and StringIO objects, but compression is not that straight-forward. It would be great for the gzip module to have "compress" and "uncompress" functions at the module level without having to go through GzipFile every-time. I think being able to send gzip compressed strings directly would be useful for applications which require to send gzip data over the wire without having to write to files and read-back. |
|
|
| msg70594 - (view) |
Author: Anand B Pillai (pythonhacker) * |
Date: 2008-08-01 21:23 |
| Uploading a file containing two functions which can be used to provide compress/uncompress on gzip. This is not a patch to gzip.py, but a stand-alone file. It this looks fine, I will make a patch. |
|
|
| msg70646 - (view) |
Author: Antoine Pitrou (pitrou) *  |
Date: 2008-08-02 23:14 |
| I think the functionality is useful, although it may be too late to make it into 2.6/3.0. Some comments: - the functions should have docstrings - there should be unit tests - style nit: named parameters in function calls (and default values in function declarations) shouldn't have spaces around the '=' sign |
|
|
| msg70650 - (view) |
Author: Anand B Pillai (pythonhacker) * |
Date: 2008-08-03 06:08 |
| Thanks. The file I uploaded was not an actual patch but just how the functions would appear in the gzip module, to illustrate the code. I can make an actual patch to gzip.py with docstrings, unit-tests and taking care of your other comments. Do you think this will be in time for Python 3.0 ? I can do this right away. |
|
|
| msg70657 - (view) |
Author: Anand B Pillai (pythonhacker) * |
Date: 2008-08-03 19:21 |
| I have uploaded the actual patch file which can be applied against the gzip.py module. I did not modify the _test() function since it is written for testing file compression. I can modify test_gzip.py if this patch is accepted. |
|
|
| msg70684 - (view) |
Author: Anand B Pillai (pythonhacker) * |
Date: 2008-08-04 04:10 |
| Uploading the svn diff. |
|
|
| msg70686 - (view) |
Author: Anand B Pillai (pythonhacker) * |
Date: 2008-08-04 04:29 |
| Added test case in test_gzip.py. Attaching svn diff of the same. |
|
|
| msg94382 - (view) |
Author: Anand B Pillai (pythonhacker) * |
Date: 2009-10-23 12:51 |
| Can we include this for the next release ? |
|
|
| msg94432 - (view) |
Author: Gregory P. Smith (gregory.p.smith) * |
Date: 2009-10-24 18:21 |
| Seems simple enough, I don't see why not. |
|
|
| msg94683 - (view) |
Author: Antoine Pitrou (pitrou) * |
Date: 2009-10-29 19:47 |
| The patch lacks a documentation update, though. (see Doc/library/gzip.rst) |
|
|
| msg94693 - (view) |
Author: Anand B Pillai (pythonhacker) * |
Date: 2009-10-30 03:59 |
| Ok, I will add this. |
|
|
| msg95020 - (view) |
Author: Anand B Pillai (pythonhacker) * |
Date: 2009-11-07 17:41 |
| Uploading fresh gzip.py SVN diff with a minor change, i.e accepting a regular string as argument, aka zlib.compress. |
|
|
| msg95021 - (view) |
Author: Anand B Pillai (pythonhacker) * |
Date: 2009-11-07 17:41 |
| Uploading rst documentation svn diff for module. |
|
|
| msg110680 - (view) |
Author: Mark Lawrence (BreamoreBoy) * |
Date: 2010-07-18 20:13 |
| This appears to have slipped under the radar, seems to sum it up. Could someone please take this forward. |
|
|
| msg114170 - (view) |
Author: Antoine Pitrou (pitrou) * |
Date: 2010-08-17 21:13 |
| Modified patch committed in r84155 (py3k). Thanks for your contribution! |
|
|
| msg117797 - (view) |
Author: Anand B Pillai (pythonhacker) * |
Date: 2010-10-01 13:23 |
| Okay. Verified as working. Thank you! |
|
|