libosmocore[master]: implement support for 3-digit MNC with leading zeros
gerrit-no-reply at lists.osmocom.org
Mon Feb 26 23:19:28 UTC 2018
Patch Set 4:
> Ok, agreed. However, for new API functions introduced, like
> gsm0808_create_layer3_aoip2 in your patch, I would like to see them
> take an aggregate structure as input, rather than an endless list
> of function arguments. If we create a new function/symbol, we
> don't have to hand mcc/mnc/lac/ci/... as individual arguments
> anymore. Does that make sense?
Yes ok, makes sense. And btw, how about dropping the _aoip? If it's going to be a general function for AoIP and non-AoIP depending on arg being NULL or not, I guess I'll rather not call it _aoip.
Also osmo_cell_global_id is a good pick, I was just reminded of it looking at stsp's cell identifier list patch. We seem to have some separate ecosystems for all of the PLMN/LAC/RAC/CI things :) osmo_cell_global_id and its siblings seem the most sensible ones...
To view, visit https://gerrit.osmocom.org/6659
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
More information about the gerrit-log