Add inspect.unwrap(f) to easily unravel "__wrapped__" chains

[eric.araujo]

http://bugs.python.org/review/13266/diff/4751/16851
File Lib/inspect.py (right):

http://bugs.python.org/review/13266/diff/4751/16851#newcode1232
Lib/inspect.py:1232: raise ValueError('wrapper loop')
Idea: include the func name in the error message

[aliles]

Reviewed changes. Needed to make minor changes to the patches to get them to
apply to the current trunk, nothing significant.

http://bugs.python.org/review/13266/diff/4751/Lib/inspect.py
File Lib/inspect.py (right):

http://bugs.python.org/review/13266/diff/4751/Lib/inspect.py#newcode1221
Lib/inspect.py:1221: def unwrap(func):
Documentation for unwrap() should be added to the standard library documentation
in Doc/library/inspect.rst?

http://bugs.python.org/review/13266/diff/4751/Lib/test/test_inspect.py
File Lib/test/test_inspect.py (right):

http://bugs.python.org/review/13266/diff/4751/Lib/test/test_inspect.py#newcod...
Lib/test/test_inspect.py:1172: 
Tests confirmed to pass on current trunk after apply patch to inspect module.

[eric.araujo]

http://bugs.python.org/review/13266/diff/4751/Lib/inspect.py
File Lib/inspect.py (right):

http://bugs.python.org/review/13266/diff/4751/Lib/inspect.py#newcode1221
Lib/inspect.py:1221: def unwrap(func):
Yes please.

http://bugs.python.org/review/13266/diff/4751/Lib/test/test_inspect.py
File Lib/test/test_inspect.py (right):

http://bugs.python.org/review/13266/diff/4751/Lib/test/test_inspect.py#newcod...
Lib/test/test_inspect.py:1172: 
I assume you mean the default branch when you say trunk.  Unfortunately 3.3 is
in RC stage and new features can’t go in.

[ezio.melotti]

http://bugs.python.org/review/13266/diff/6441/Doc/library/inspect.rst
File Doc/library/inspect.rst (right):

http://bugs.python.org/review/13266/diff/6441/Doc/library/inspect.rst#newcode800
Doc/library/inspect.rst:800: Get the object wrapped by *func*. If follows the
chain of *__wrapped__*
s/If/It/ (or just "Follow ...")
*__wrapped__* -> :attr:`__wrapped__` (bonus points if it links to wherever
__wrapped__ is documented (if it is documented at all)).

http://bugs.python.org/review/13266/diff/6441/Lib/inspect.py
File Lib/inspect.py (right):

http://bugs.python.org/review/13266/diff/6441/Lib/inspect.py#newcode373
Lib/inspect.py:373: memo = {id(func)}
Why the id and not func itself?

http://bugs.python.org/review/13266/diff/6441/Lib/inspect.py#newcode377
Lib/inspect.py:377: raise ValueError('wrapper loop')
This error could indeed be improved to include the function name.

http://bugs.python.org/review/13266/diff/6441/Lib/test/test_inspect.py
File Lib/test/test_inspect.py (right):

http://bugs.python.org/review/13266/diff/6441/Lib/test/test_inspect.py#newcod...
Lib/test/test_inspect.py:2300: self.assertIs(inspect.unwrap(C()), None)
You can use assertIsNone here.
I would also test with some other decorators of the stdlib.

[durban]

http://bugs.python.org/review/13266/diff/6441/Lib/inspect.py
File Lib/inspect.py (right):

http://bugs.python.org/review/13266/diff/6441/Lib/inspect.py#newcode373
Lib/inspect.py:373: memo = {id(func)}
On 2012/11/04 05:23:39, ezio.melotti wrote:
> Why the id and not func itself?

Because func may not by hashable of may not have an identity hash.

http://bugs.python.org/review/13266/diff/6441/Lib/test/test_inspect.py
File Lib/test/test_inspect.py (right):

http://bugs.python.org/review/13266/diff/6441/Lib/test/test_inspect.py#newcod...
Lib/test/test_inspect.py:2300: self.assertIs(inspect.unwrap(C()), None)
On 2012/11/04 05:23:39, ezio.melotti wrote:
> I would also test with some other decorators of the stdlib.

I wasn't able to find another decorator in the stdlib that sets __wrapped__.
Could you suggest some?