Message220252
On 11.06.2014 12:32, Vinay Sajip wrote:
>
> Vinay Sajip added the comment:
>
>> _srcfile is only used to identify the caller's stack frame
>
> Not quite. It's also used to indicate whether findCaller() should be called at all: setting it to None avoids calling findCaller(), which might be desirable in some performance-sensitive scenarios.
>
> So if you mean "just call _get_module_filename() instead of accessing _srcFile", that won't do. If you mean "set _srcFile to the return value of _get_module_filename()", that might work, if I e.g. move the _srcFile definition to after addLevelName (say) and do just
>
> _srcFile = addLevelName.__code__.co_filename
>
> How does that sound?
That's what I meant, yes. Please also add some comment explaining
why this is done in this way.
FWIW: Given that __file__ is not always set, it may be worthwhile
introducing some generic helper to the stdlib which uses the
.co_filename attribute to get the compile time filename as fallback
in case __file__ is not set.
Thanks,
--
Marc-Andre Lemburg
eGenix.com |
|
Date |
User |
Action |
Args |
2014-06-11 10:45:52 | lemburg | set | recipients:
+ lemburg, vinay.sajip, python-dev |
2014-06-11 10:45:52 | lemburg | link | issue21709 messages |
2014-06-11 10:45:52 | lemburg | create | |
|