Stefan Behnel added the comment:

1) that's a regex test, so it just looks for the text. I only extended it to 
include the object it's supposed to test for. It's not actually related to my 
changes, but I considered it the right thing to do. I used "42.*is not" in 
order to allow for alternative spellings like "'42' is not...".

2) "raise ... from None" is the official way to explicitly drop the exception 
context from a newly raised exception. Otherwise, you'd get two exceptions in 
this case: a TypeError (as main exception) and an AttributeError (as its 
context), which I do not consider helpful here as it bloats the output for an 
otherwise simple error. For 99% of the use cases, it won't matter which 
attribute was missing.

----------

_______________________________________
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

Reply via email to