Fix/recursion limit

[swaradgat19]

fixes #192

[kaczmarj]

there's a typing error:

wsinfer/patchlib/patch.py:20: error: Argument 1 to "contextmanager" has incompatible type "Callable[[int], None]"; expected "Callable[[int], Iterator[Never]]"  [arg-type]
wsinfer/patchlib/patch.py:21: error: The return type of a generator function should be "Generator" or one of its supertypes  [misc]
wsinfer/patchlib/patch.py:24: error: "setrecursionlimit" does not return a value (it only ever returns None)  [func-returns-value]

try running mypy locally to catch this problem. i believe the return type should be Iterator[None].

from typing import Iterator

# .....

@contextmanager
def temporary_recursion_limit(limit: int) -> Iterator[None]:
    old_limit = sys.getrecursionlimit()
    try:
        yield sys.setrecursionlimit(limit)
    finally:
        sys.setrecursionlimit(old_limit)

[swaradgat19]

@kaczmarj I'm able to work around the issue by writing the yield after setting the recursion limit.

@contextmanager
def temporary_recursion_limit(limit: int) -> Iterator[None]:
    old_limit = sys.getrecursionlimit()
    try:
        sys.setrecursionlimit(limit) 
        yield
    finally:
        sys.setrecursionlimit(old_limit)

[kaczmarj]

@kaczmarj I'm able to work around the issue by writing the yield after setting the recursion limit.

@contextmanager
def temporary_recursion_limit(limit: int) -> Iterator[None]:
    old_limit = sys.getrecursionlimit()
    try:
        sys.setrecursionlimit(limit) 
        yield
    finally:
        sys.setrecursionlimit(old_limit)

looks great!

[swaradgat19]

@kaczmarj looks good!

[kaczmarj]

thanks @swaradgat19 !