Message273333
Another argument for having the fix in `unwrap` rather than `signature` is that this bug does not actually seem to be called by `signature`, as the doctest module calls `unwrap` for "inspect.isroutine(inspect.unwrap(val))".
Also, this call does not even check for `ValueError`, which, if I'm not wrong, is something that should be corrected.
Maybe `unwrap` could be made recursive to make it respect recursion limits directly ? Otherwise, limiting the loop seems like a good idea.
(Temporarily, `from mock import call; call.__wrapped__ = None` seems to be a good workaround to prevent infinite memory allocation). |
|
Date |
User |
Action |
Args |
2016-08-22 04:40:39 | pstch | set | recipients:
+ pstch, ncoghlan, cjw296, michael.foord, yselivanov, The Compiler |
2016-08-22 04:40:38 | pstch | set | messageid: <1471840838.98.0.167349237412.issue25532@psf.upfronthosting.co.za> |
2016-08-22 04:40:38 | pstch | link | issue25532 messages |
2016-08-22 04:40:38 | pstch | create | |
|