Attention is currently required from: kirr.

pespin has posted comments on this change by kirr. ( https://gerrit.osmocom.org/c/osmocom-bb/+/39536?usp=email )

Change subject: trx_toolkit/*: Represent bursts as arrays instead of lists ......................................................................

Patch Set 1: Code-Review+1

(4 comments)

Commit Message:

https://gerrit.osmocom.org/c/osmocom-bb/+/39536/comment/9f680289_2b3a4fd7?us... : PS1, Line 13: item as just byte, and by leveraging bytearray.translate, we can sped sped -> speed up?

File src/target/trx_toolkit/data_msg.py:

https://gerrit.osmocom.org/c/osmocom-bb/+/39536/comment/5c4d3857_f9657b3c?us... : PS1, Line 71: self.burst = burst # bytes|bytearray for ubit, array[b] for sbit, array[B] for usbit I don't really understand what you mean with this comment here.

https://gerrit.osmocom.org/c/osmocom-bb/+/39536/comment/aae995a1_6d58e490?us... : PS1, Line 134: def usbit2sbit(bits): # array[B] -> array[b] sou one cannot declare a param type for an array as done previously by List?

https://gerrit.osmocom.org/c/osmocom-bb/+/39536/comment/55548750_3c79688e?us... : PS1, Line 672: burst = array('B', burst) I wonder whether we'll probably want to have all this encapsulated into eg. SBits, UBits, etc. objects in the future, so that it does all this array mangling internally and conversion to other objects internally in the class.