Change in osmo-bsc[master]: abis_rsl: print_meas_rep_{buf, uni}() accept const *mr

[fixeria]

fixeria 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>