osmo-gsm-tester[master]: log: Use sys._getframe() to avoid stat(2) calls

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Holger Freyther gerrit-no-reply at lists.osmocom.org
Tue Feb 13 09:29:32 UTC 2018


Patch Set 2:

(1 comment)

https://gerrit.osmocom.org/#/c/6377/1/src/osmo_gsm_tester/log.py
File src/osmo_gsm_tester/log.py:

Line 278:     # Poke into internal to avoid hitting the linecache which will make one or
> According to [1], _getframe() can have a depth parameter, which if I unders
Thanks for looking up the docs. I assumed it would not be documented at all.

I wouldn't want to catch AttributeError.
The code will never be executed (and hence likely broken or at least bitrotting). Then PyPy supports sys._getframe() as well (https://bitbucket.org/pypy/pypy/src/fc85b98a24e2a5a30d667659260089db372817f1/pypy/module/sys/__init__.py?at=default&fileviewer=file-view-default#__init__.py-55)

ValueError:
The unknown/unknown was just because I iterate myself. I think getting a ValueError is good. If somebody passes a bogus offset we can discover this quickly (instead of silencing it).


-- 
To view, visit https://gerrit.osmocom.org/6377
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I8f6bacadcf74d3aa25db1e1f41644f64aa19cf92
Gerrit-PatchSet: 2
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Owner: Holger Freyther <holger at freyther.de>
Gerrit-Reviewer: Holger Freyther <holger at freyther.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Reviewer: neels <nhofmeyr at sysmocom.de>
Gerrit-HasComments: Yes



More information about the gerrit-log mailing list