Julian Berman <[email protected]> added the comment:
For reference, the implementation that I posted in the other thread is:
@contextlib.contextmanager
def maybe(got, contextfactory, *args, checkif=bool, **kwargs):
if checkif(got):
yield got
else:
with contextfactory(*args, **kwargs) as got:
yield got
which produces code like:
def fn(input_file=None):
with maybe(input_file, open, "/default/input/file/location/"):
for line in input_file:
print line
For the example given here in the OP, since rmtree isn't a contextmanager it'd
require wrapping it in one first.
I could probably understand if there was desire for these to be a recipe
instead. The advantage (if any) of the above over this class is that it keeps
context managers sorta looking like context managers. Here if you wanted to
write my example it'd require writing code like
with ContextManager() as mgr:
foo = ctxtmgr()
mgr.register(foo.__close__)
which doesn't look too great :/
The class though does have wider scope though, since it doesn't necessarily
only need a context manager, it can do cleanup for anything, which I guess is
nice.
----------
nosy: +Julian
_______________________________________
Python tracker <[email protected]>
<http://bugs.python.org/issue13585>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
