Martin Panter added the comment:

Thanks for the patches. This test was added in r74435, presumably to test for a 
crash. IMO it would be sufficient to test for the first part of the error 
message, without checking the column number. The test could also use 
assertRaisesRegex().

Should this actually be applied to all security versions, or just 3.5+ and 2.7? 
I guess can see the benefit of fixing 3.2+, and I remember doing this once 
before when the test suite fell over and it was not Python’s fault.

----------
nosy: +martin.panter
stage:  -> patch review

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

Reply via email to