kirr has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmocom-bb/+/40053?usp=email )
Change subject: trx_toolkit/burst_fwd: Do not inherit BurstForwarder from TRXList ......................................................................
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