Jonas Wagner <ve...@gmx.ch> added the comment:

Hi Tal,

Thanks a lot for your feedback.

My primary objective was to increase the test coverage for cgi.py. If it is a 
problem to have the additional tests in this patch I'm happy to create a new 
issue with a separate patch. 

The default value for the boundary was an oversight, sorry for that.

You are right regarding the commented out boundary as well, I forgot to refresh 
the patch. Again, sorry.

Do you think valid_boundary should contain a check to ensure it is a byte 
object?

----------
Added file: http://bugs.python.org/file22567/cgi-coverage-2.diff

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue12411>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to