[Python-Dev] [Python-checkins] cpython (merge 3.2 -> default): Merge with 3.2.
Victor Stinner
victor.stinner at haypocalc.com
Thu Jul 14 22:33:09 CEST 2011
Done:
changeset: 71337:66e519792e4c
tag: tip
user: Victor Stinner <victor.stinner at haypocalc.com>
date: Thu Jul 14 22:28:36 2011 +0200
files: Lib/cgi.py Lib/test/test_cgi.py Misc/NEWS
description:
Add cgi.closelog() function to close the log file
Le 14/07/2011 14:57, Ezio Melotti a écrit :
>
>> diff --git a/Lib/test/test_cgi.py b/Lib/test/test_cgi.py
>> --- a/Lib/test/test_cgi.py
>> +++ b/Lib/test/test_cgi.py
>> @@ -155,7 +155,13 @@
>> cgi.logfp = None
>> cgi.logfile = "/dev/null"
>> cgi.initlog("%s", "Testing log 3")
>> - self.addCleanup(cgi.logfp.close)
>> + def log_cleanup():
>> + """Restore the global state of the log vars."""
>> + cgi.logfile = ''
>> + cgi.logfp.close()
>> + cgi.logfp = None
>> + cgi.log = cgi.initlog
>
>
> It was suggested (on #python-dev) to move this function to the cgi
> module itself, but since I'm not familiar with it I just added it here
> in order to fix a failure in the test.
>
> The cgi module has two global vars (logfile and logfp) and a global
> function (log) that is initialized to initlog and then reassigned to
> either dolog or nolog (a dummy function that does nothing) in initlog
> itself[0].
>
> If someone thinks the log_cleanup function should be moved to the
> cgi.py module and/or the code be refactored a bit, he can do it or
> open an issue.
>
> [0]: http://hg.python.org/cpython/file/ac1c3291a689/Lib/cgi.py#l50
>
> Best Regards,
More information about the Python-Dev
mailing list