filesystem: Use Tuple for transparent file size
As the documentation strings say: The size should be a *tuple*. In
reality we so far passed a set. The problem with the set is that
ordering is not guaranteed, and hence we cannot assume the first and
second item have meaning (minimum vs. default size).
While at it, use a type annotation to catch such bugs easily.
Change-Id: I553616f8c6c4aaa8f635b3d7d94e8e8f49ed5a56
diff --git a/pySim/gsm_r.py b/pySim/gsm_r.py
index b48469e..ccab87f 100644
--- a/pySim/gsm_r.py
+++ b/pySim/gsm_r.py
@@ -149,7 +149,7 @@
"""Section 7.3"""
def __init__(self):
- super().__init__(fid='6ff2', sfid=None, name='EF.CallconfC', size={24, 24},
+ super().__init__(fid='6ff2', sfid=None, name='EF.CallconfC', size=(24, 24),
desc='Call Configuration of emergency calls Configuration')
self._construct = Struct('pl_conf'/PlConfAdapter(Int8ub),
'conf_nr'/BcdAdapter(Bytes(8)),
@@ -183,7 +183,7 @@
def __init__(self):
super().__init__(fid='6ff4', sfid=None,
- name='EF.Shunting', desc='Shunting', size={8, 8})
+ name='EF.Shunting', desc='Shunting', size=(8, 8))
self._construct = Struct('common_gid'/Int8ub,
'shunting_gid'/HexAdapter(Bytes(7)))
@@ -253,7 +253,7 @@
"""Section 8.6"""
def __init__(self, fid, name, desc):
- super().__init__(fid=fid, sfid=None, name=name, desc=desc, size={4, 4})
+ super().__init__(fid=fid, sfid=None, name=name, desc=desc, size=(4, 4))
self._construct = Struct('next_table_type'/NextTableType,
'id_of_next_table'/HexAdapter(Bytes(2)),
'dialed_digits'/BcdAdapter(Bytes(1)))