mirror of
https://github.com/zulip/zulip.git
synced 2025-11-19 14:08:23 +00:00
profile: Strengthen decorator types using ParamSpec.
Signed-off-by: Zixuan James Li <359101898@qq.com>
This commit is contained in:
committed by
Tim Abbott
parent
e632a4ced2
commit
532f827a7f
@@ -1,11 +1,14 @@
|
|||||||
import cProfile
|
import cProfile
|
||||||
from functools import wraps
|
from functools import wraps
|
||||||
from typing import Callable, TypeVar, cast
|
from typing import Callable, TypeVar
|
||||||
|
|
||||||
FuncT = TypeVar("FuncT", bound=Callable[..., object])
|
from typing_extensions import ParamSpec
|
||||||
|
|
||||||
|
ParamT = ParamSpec("ParamT")
|
||||||
|
ReturnT = TypeVar("ReturnT")
|
||||||
|
|
||||||
|
|
||||||
def profiled(func: FuncT) -> FuncT:
|
def profiled(func: Callable[ParamT, ReturnT]) -> Callable[ParamT, ReturnT]:
|
||||||
"""
|
"""
|
||||||
This decorator should obviously be used only in a dev environment.
|
This decorator should obviously be used only in a dev environment.
|
||||||
It works best when surrounding a function that you expect to be
|
It works best when surrounding a function that you expect to be
|
||||||
@@ -22,14 +25,13 @@ def profiled(func: FuncT) -> FuncT:
|
|||||||
./tools/show-profile-results test_ratelimit_decrease.profile
|
./tools/show-profile-results test_ratelimit_decrease.profile
|
||||||
|
|
||||||
"""
|
"""
|
||||||
func_: Callable[..., object] = func # work around https://github.com/python/mypy/issues/9075
|
|
||||||
|
|
||||||
@wraps(func)
|
@wraps(func)
|
||||||
def wrapped_func(*args: object, **kwargs: object) -> object:
|
def wrapped_func(*args: ParamT.args, **kwargs: ParamT.kwargs) -> ReturnT:
|
||||||
fn = func.__name__ + ".profile"
|
fn = func.__name__ + ".profile"
|
||||||
prof = cProfile.Profile()
|
prof = cProfile.Profile()
|
||||||
retval = prof.runcall(func_, *args, **kwargs)
|
retval = prof.runcall(func, *args, **kwargs)
|
||||||
prof.dump_stats(fn)
|
prof.dump_stats(fn)
|
||||||
return retval
|
return retval
|
||||||
|
|
||||||
return cast(FuncT, wrapped_func) # https://github.com/python/mypy/issues/1927
|
return wrapped_func
|
||||||
|
|||||||
Reference in New Issue
Block a user