Nir Soffer has posted comments on this change.
Change subject: assert: Replace assetion with AssertionError ......................................................................
Patch Set 3:
(1 comment)
http://gerrit.ovirt.org/#/c/29307/3//COMMIT_MSG Commit Message:
Line 5: CommitDate: 2014-10-12 20:49:15 +0300 Line 6: Line 7: assert: Replace assetion with AssertionError Line 8: Line 9: If Python is run with -O flag, assertions are removed. This may casue a
Is this a real problem? If someone takes the trouble to run with -O, I'd s
This was requested by Federico, and I agree that this is more correct. I'm not suggesting to run with -O. Line 10: useful error message to be hidden, making it harder to debug, or worse, Line 11: code invoke inside an assert will not run, changing the flow of the Line 12: "optimized" code. Line 13: