Message202998
Since failed asserts print the failed assert, repeating the assertion in a message is useless.
>>> assert 1 <= i
Traceback (most recent call last):
File "<pyshell#3>", line 1, in <module>
assert 1 <= i
AssertionError
It is already obvious that i must be >= 1. So I would reject the patch.
> And isn't this too much defensive programming?
Whether stdlib python code should have asserts is a more interesting question. I will ask on pydev. |
|
Date |
User |
Action |
Args |
2013-11-16 00:57:30 | terry.reedy | set | recipients:
+ terry.reedy, thatiparthy |
2013-11-16 00:57:30 | terry.reedy | set | messageid: <1384563450.45.0.729620584898.issue19562@psf.upfronthosting.co.za> |
2013-11-16 00:57:30 | terry.reedy | link | issue19562 messages |
2013-11-16 00:57:30 | terry.reedy | create | |
|