Change in osmo-bsc[master]: rsl_lchan_lookup(): turn cbits if-cascade into a switch()

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

neels gerrit-no-reply at lists.osmocom.org
Mon May 31 05:39:59 UTC 2021


Hello Jenkins Builder, laforge, pespin, 

I'd like you to reexamine a change. Please visit

    https://gerrit.osmocom.org/c/osmo-bsc/+/24428

to look at the new patch set (#8).

Change subject: rsl_lchan_lookup(): turn cbits if-cascade into a switch()
......................................................................

rsl_lchan_lookup(): turn cbits if-cascade into a switch()

There is some odd repetition of macros with all possible subslot
indexes, but now we are using the defined macros instead of magic
numbers, making it easier to see that it is correct.

Also this allows trivially adding VAMOS cbits cases to the existing
switch() in a subsequent patch.

Change-Id: I4c6ce2f4ca999c19a58909b9adb849a3b8b6ffc8
---
M src/osmo-bsc/bts_trx.c
1 file changed, 28 insertions(+), 7 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/28/24428/8
-- 
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/24428
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I4c6ce2f4ca999c19a58909b9adb849a3b8b6ffc8
Gerrit-Change-Number: 24428
Gerrit-PatchSet: 8
Gerrit-Owner: neels <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210531/98f39ceb/attachment.htm>


More information about the gerrit-log mailing list