Author serhiy.storchaka
Recipients Martin.Teichmann, ncoghlan, serhiy.storchaka
Date 2017-05-09.09:12:22
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1494321144.29.0.525652304123.issue30306@psf.upfronthosting.co.za>
In-reply-to
Content
Wouldn't be better to split _GeneratorContextManager on two classes rather than add a boolean argument? Currently _GeneratorContextManager used  two different functions -- an one-shot context manager and a decorator that recreates context managers.

Proposed patch makes _GeneratorContextManager an one-shot context manager again (it no longer keep references to arguments), and adds _GeneratorContextManagerDecorator which can be used either as an one-shot context manager (references to arguments are removed after use) or as a decorator (in the last case the gen attribute is not created).
History
Date User Action Args
2017-05-09 09:12:24serhiy.storchakasetrecipients: + serhiy.storchaka, ncoghlan, Martin.Teichmann
2017-05-09 09:12:24serhiy.storchakasetmessageid: <1494321144.29.0.525652304123.issue30306@psf.upfronthosting.co.za>
2017-05-09 09:12:24serhiy.storchakalinkissue30306 messages
2017-05-09 09:12:23serhiy.storchakacreate