bitfrost assertion checks were lost
James Cameron
quozl at laptop.org
Tue Jul 6 19:32:43 EDT 2010
On Tue, Jul 06, 2010 at 12:27:24PM -0600, Daniel Drake wrote:
> A few months ago we enabled Python optimizations in the OLPC builds
> causing python's "assert" statement to do nothing.
> Martin Langhoff pointed out that we're dropping some important parts
> of bitfrost's code with this change - we do want to be making these
> checks.
> Here's a patch to restore the previous behaviour, review appreciated.
Since the git commit ea73e4a was slightly later, I reviewed that. No
issues found. I've also tested to my satisfaction that assert(False)
does nothing when python -O is run, and raising AssertionError is still
possible.
Reviewed-by: James Cameron <quozl at laptop.org>
--
James Cameron
http://quozl.linux.org.au/
More information about the Devel
mailing list