2010/8/9 Nick Coghlan <ncogh...@gmail.com>: > On Mon, Aug 9, 2010 at 11:05 PM, benjamin.peterson > <python-check...@python.org> wrote: >> -if hasattr(sys, '_getframe'): >> - currentframe = sys._getframe >> -else: >> - currentframe = lambda _=None: None >> +def currentframe(): >> + """Return the frame or the caller or None if this is not possible.""" >> + return sys._getframe(1) if hasattr(sys, "_getframe") else None > > It isn't hugely important, but with sys._getframe() unlikely to appear > during runtime, the following may make more sense: > > if hasattr(sys, '_getframe'): > def currentframe(): > return sys._getframe(1) > else: > def currentframe(): > pass > currentframe.__doc__ = "Return the frame of the caller or None if this > is not possible."
I considered that but found the docstring thing ugly. > > (Oh, and there's a typo in the docstring...) Thanks. -- Regards, Benjamin _______________________________________________ Python-Dev mailing list Python-Dev@python.org http://mail.python.org/mailman/listinfo/python-dev Unsubscribe: http://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com