kirr has uploaded this change for review.

View Change

trx_toolkit/burst_fwd: Do not inherit BurstForwarder from TRXList

That is simply unneeded as TRXList is used primarly by
fake_trx.Application when doing initial setup of TRX list, while
for BurstForwarder it is enough to have just a regular list of TRX
instances.

We need to remove TRXList from BurstForwarder base to be able to switch
BurstForwarder to cdef class in a follow-up patch as that is possible
only with cdef class basesm, while TRXList is plain py class.

Change-Id: I9ce8ca1531eea5d59358fb41cc99d8c349757abb
---
M src/target/trx_toolkit/burst_fwd.pyx
1 file changed, 4 insertions(+), 3 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/53/40053/1
diff --git a/src/target/trx_toolkit/burst_fwd.pyx b/src/target/trx_toolkit/burst_fwd.pyx
index 2737232..36e5095 100644
--- a/src/target/trx_toolkit/burst_fwd.pyx
+++ b/src/target/trx_toolkit/burst_fwd.pyx
@@ -21,9 +21,7 @@

import logging as log

-from trx_list import TRXList
-
-class BurstForwarder(TRXList):
+class BurstForwarder:
""" Performs burst forwarding between transceivers.

BurstForwarder distributes bursts between the list of given
@@ -42,6 +40,9 @@

"""

+ def __init__(self, trx_list):
+ self.trx_list = trx_list
+
def forward_msg(self, src_trx, rx_msg):
# Originating Transceiver may use frequency hopping,
# so let's precalculate its Tx frequency in advance

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

Gerrit-MessageType: newchange
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I9ce8ca1531eea5d59358fb41cc99d8c349757abb
Gerrit-Change-Number: 40053
Gerrit-PatchSet: 1
Gerrit-Owner: kirr <kirr@nexedi.com>
Gerrit-CC: fixeria <vyanitskiy@sysmocom.de>
Gerrit-CC: osmith <osmith@sysmocom.de>
Gerrit-CC: pespin <pespin@sysmocom.de>