-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Matthew Wilkes wrote: > Log message for revision 93717: > assert isn't a function, using parens will cause the two arguments to be > treated as a 2-tuple, hence always true. > > Changed: > U zope.tal/trunk/src/zope/tal/dummyengine.py > > -=- > Modified: zope.tal/trunk/src/zope/tal/dummyengine.py > =================================================================== > --- zope.tal/trunk/src/zope/tal/dummyengine.py 2008-12-06 12:09:53 UTC > (rev 93716) > +++ zope.tal/trunk/src/zope/tal/dummyengine.py 2008-12-06 13:25:25 UTC > (rev 93717) > @@ -85,8 +85,8 @@ > return value > > def evaluate(self, expression): > - assert (expression.startswith("$") and expression.endswith("$"), > - expression) > + assert expression.startswith("$") and expression.endswith("$"), \ > + expression > expression = expression[1:-1] > m = name_match(expression) > if m: > @@ -152,8 +152,8 @@ > return self.evaluate(expr) > > def evaluateMacro(self, macroName): > - assert (macroName.startswith("$") and macroName.endswith("$"), > - macroName) > + assert macroName.startswith("$") and macroName.endswith("$"), \ > + macroName > macroName = macroName[1:-1] > file, localName = self.findMacroFile(macroName) > if not file:
A better fix would be to strip outthe 'assert' keyword everywhere, and use 'self.failUnless' / 'self.failIf' instead: that would allow getting rid of the "backsplash", as well. Tres. - -- =================================================================== Tres Seaver +1 540-429-0999 [EMAIL PROTECTED] Palladion Software "Excellence by Design" http://palladion.com -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.6 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org iD8DBQFJPVjG+gerLs4ltQ4RAjZKAKDSJ2alTo+X6JjUypulCCB1cryn3QCfUktE Unhlfp28gYNPB3pyyHl+iM4= =n6K5 -----END PGP SIGNATURE----- _______________________________________________ Zope-Dev maillist - Zope-Dev@zope.org http://mail.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - http://mail.zope.org/mailman/listinfo/zope-announce http://mail.zope.org/mailman/listinfo/zope )