Include "argument-resolving decorator" into decorator package ? about decorator HOT 2 CLOSED

You should make your case better, I am not understanding what you are proposing. The decorator module has never been a collection of useful decorators, so it is really hard to have something added to it. Not to see that any extension is out of question, but you should convince me that this is general enough to be included.

from decorator.

Actually if I see well the "resolving_decorator" included in that package, it's because imho it's just a variant of the "decorator.decorator" utility.

Below is an excerpt from the decorator documentation.

>>> @decorator
... def trace(f, *args, **kw):
... kwstr = ', '.join('%r: %r' % (k, kw[k]) for k in sorted(kw))
... print("calling %s with args %s, {%s}" % (f.__name__, args, kwstr))
... return f(*args, **kw)

As you see, when the decorated function get called at runtime, all arguments are passed as standard "_args, *_kw" pair. So inside this "trace" decorator, it's complicated to extract, for example, an argument which would be an SQL session, because one doesn't know if it's in "args" (passed by position) or "kwargs" (passed by keyword).

The "resolving_decorator" would allow "trace" to receive a single "**all_kwargs" data set, with all arguments already bound to their final "local variable" name. Thus "trace" could simply do all_kwargs.get("sql_session"), and fiddle with this pre-resolved parameter. The only constraint would be, of course, that all decorated functions which want to have sql session handling, expose a "sql_session" argument in their signature.

from decorator.