Attention is currently required from: pespin.
4 comments:
File include/osmocom/bsc/gsm_data.h:
Patch Set #6, Line 657: uint16_t parameter[_NUM_NM_RLC_PAR_OFFSET];
why changing the name? I don't see the point in adding _OFFSET it just makes stuff larger with no be […]
I added the suffix, because this makes clear (self-documenting code and such) that their values aren't to be touched, since these values are used as offsets.
The rest of the name change went along with moving the enums from separate headers (iirc all with basename `gsm_data.h`) in `osmo-bsc`/`osmo-bts` (and possibly `osmo-pcu`) to `libosmocore.git:include/osmocom/gsm/protocol/gsm_12_21.h` to avoid duplicating the same enum values across multiple projects when it makes more sense to have them centrally in a library.
File include/osmocom/bsc/gsm_data.h:
Patch Set #1, Line 654: enum gprs_cs {
IIUC the one under gsm_44_06. […]
ah yes they have different values and I grepped for the bitmask; it isn't always accessed by use of the enum identifiers either
File src/osmo-bsc/bts_ipaccess_nanobts_omlattr.c:
Patch Set #6, Line 173: for (offset = NM_RLC_T3142_OFFSET; offset <= NM_CV_COUNTDOWN_OFFSET;
why not simply memcpy?
Tbh. we don't have to have a for loop here, I don't care much either way and it might be better to keep it the way it was: Line by line assignments. I have discarded the loop usage.
Makes the patch easier to read as well since it's focussing on the core change.
`memcpy()` doesn't make much sense since the source is an array of `uint16_t` and the destination is a regular byte-array.
Patch Set #6, Line 174: ++offset)
please don't use preincrement unless it really has a purpose over postincrement (code convention in […]
Alright I'll keep it in mind, thank you
To view, visit change 31304. To unsubscribe, or for help writing mail filters, visit settings.