laforge submitted this change.

View Change


Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified
runtime: cosmetic: prnounce file reference data

One of the most important properties of the RuntimeLchan are the
selected_file/adf properties. Let's reformat the code so that those
properties are more pronounced.

Change-Id: I4aa028f66879b7d6c2a1cd102cda8d8ca5ff48b1
Related: OS#5418
---
M pySim/runtime.py
1 file changed, 17 insertions(+), 1 deletion(-)

diff --git a/pySim/runtime.py b/pySim/runtime.py
index 670babc..028bef1 100644
--- a/pySim/runtime.py
+++ b/pySim/runtime.py
@@ -170,11 +170,13 @@
def __init__(self, lchan_nr: int, rs: RuntimeState):
self.lchan_nr = lchan_nr
self.rs = rs
+ self.scc = self.rs.card._scc.fork_lchan(lchan_nr)
+
+ # File reference data
self.selected_file = self.rs.mf
self.selected_adf = None
self.selected_file_fcp = None
self.selected_file_fcp_hex = None
- self.scc = self.rs.card._scc.fork_lchan(lchan_nr)

def add_lchan(self, lchan_nr: int) -> 'RuntimeLchan':
"""Add a new logical channel from the current logical channel. Just affects

To view, visit change 35250. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I4aa028f66879b7d6c2a1cd102cda8d8ca5ff48b1
Gerrit-Change-Number: 35250
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-MessageType: merged