Patch Set 2:

(1 comment)
File src/osmo_gsm_tester/

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 

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
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I8f6bacadcf74d3aa25db1e1f41644f64aa19cf92
Gerrit-PatchSet: 2
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Owner: Holger Freyther <>
Gerrit-Reviewer: Holger Freyther <>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Pau Espin Pedrol <>
Gerrit-Reviewer: neels <>
Gerrit-HasComments: Yes

Reply via email to