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/.
fixeria gerrit-no-reply at lists.osmocom.orgfixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/26146 ) Change subject: abis_rsl: print_meas_rep_{buf,uni}() accept const *mr ...................................................................... abis_rsl: print_meas_rep_{buf,uni}() accept const *mr Change-Id: I5c3e27a00cd84f102558499072965ec538f5a87f --- M src/osmo-bsc/abis_rsl.c 1 file changed, 7 insertions(+), 5 deletions(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve neels: Looks good to me, approved diff --git a/src/osmo-bsc/abis_rsl.c b/src/osmo-bsc/abis_rsl.c index f849d27..0291224 100644 --- a/src/osmo-bsc/abis_rsl.c +++ b/src/osmo-bsc/abis_rsl.c @@ -1168,7 +1168,9 @@ return 0; } -static void print_meas_rep_uni(struct osmo_strbuf *sb, struct gsm_meas_rep_unidir *mru, const char *prefix) +static void print_meas_rep_uni(struct osmo_strbuf *sb, + const struct gsm_meas_rep_unidir *mru, + const char *prefix) { OSMO_STRBUF_PRINTF(*sb, "RXL-FULL-%s=%3ddBm RXL-SUB-%s=%3ddBm ", prefix, rxlev2dbm(mru->full.rx_lev), @@ -1177,7 +1179,7 @@ prefix, mru->full.rx_qual, prefix, mru->sub.rx_qual); } -static int print_meas_rep_buf(char *buf, size_t len, struct gsm_meas_rep *mr) +static int print_meas_rep_buf(char *buf, size_t len, const struct gsm_meas_rep *mr) { struct osmo_strbuf sb = { .buf = buf, .len = len }; @@ -1212,14 +1214,14 @@ return sb.chars_needed; } -static char *print_meas_rep_c(void *ctx, struct gsm_meas_rep *mr) +static char *print_meas_rep_c(void *ctx, const struct gsm_meas_rep *mr) { /* A naive count of required characters gets me to ~200, so 256 should be safe to get a large enough buffer on * the first time. */ OSMO_NAME_C_IMPL(ctx, 256, "ERROR", print_meas_rep_buf, mr) } -static void print_meas_rep(struct gsm_lchan *lchan, struct gsm_meas_rep *mr) +static void print_meas_rep(struct gsm_lchan *lchan, const struct gsm_meas_rep *mr) { int i; const char *name = ""; @@ -1240,7 +1242,7 @@ if (mr->num_cell != 7 && log_check_level(DMEAS, LOGL_DEBUG)) { for (i = 0; i < mr->num_cell; i++) { - struct gsm_meas_rep_cell *mrc = &mr->cell[i]; + const struct gsm_meas_rep_cell *mrc = &mr->cell[i]; DEBUGP(DMEAS, "IDX=%u ARFCN=%u BSIC=%u => %d dBm\n", mrc->neigh_idx, mrc->arfcn, mrc->bsic, rxlev2dbm(mrc->rxlev)); } -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/26146 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I5c3e27a00cd84f102558499072965ec538f5a87f Gerrit-Change-Number: 26146 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy at sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy at sysmocom.de> Gerrit-Reviewer: neels <nhofmeyr at sysmocom.de> Gerrit-Reviewer: pespin <pespin at sysmocom.de> Gerrit-MessageType: merged -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20211108/37a126e5/attachment.htm>