Terry J. Reedy added the comment:
I am puzzled by this part of the patch:
- with self.assertRaisesRegex(TypeError, 'is not a callable object'):
+ with self.assertRaisesRegex(TypeError, '42.*is not a callable object'):
With 3.3, I get "TypeError: 42 is not a callable object", which I consider
correct, and which should not be changed by the two changes to signature(). So
both the old test, missing '42' (how does it pass?) and the new version, with
extraneous '.*' look wrong.
I am also puzzled by the 'from None' part in
+ raise TypeError("'{!r}' is not a Python function".format(func)) from None
While I remember that being in the pydev discussion and while "raise XyzError
from None" executes, it does not seems to be documented for 3.3 in 7.8. The
raise statement. (Should this be another issue?) In fact, that section says "
if given, the second expression must be another exception class or instance",
which excludes None.
----------
_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue17159>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
