laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-gbproxy/+/22653 )
Change subject: Simplify raid data storage ...................................................................... Patch Set 2: I think this mixes two orthogonal topics: a) moving from a binary to a decoded ra_id b) reducing redundancy between the multiple places of storage "a" is uncontroversial to me. "b" is dangerous, as without detailed review I would not be sure if bvc->cell always exists in all situations where we want to read or write the ra_id. -- To view, visit https://gerrit.osmocom.org/c/osmo-gbproxy/+/22653 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-gbproxy Gerrit-Branch: master Gerrit-Change-Id: Ib58b9188e3ce4bd3fdadb03f158d56b29778387c Gerrit-Change-Number: 22653 Gerrit-PatchSet: 2 Gerrit-Owner: daniel <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <[email protected]> Gerrit-Reviewer: pespin <[email protected]> Gerrit-CC: laforge <[email protected]> Gerrit-Comment-Date: Wed, 03 Feb 2021 17:56:03 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
