laforge submitted this change.

View Change



1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

Approvals: fixeria: Looks good to me, but someone else must approve pespin: Looks good to me, approved Jenkins Builder: Verified
lc15/oc2g: remove unused variables

oml.c: In function ‘l1if_rsl_chan_mod’:
oml.c:1980:22: error: unused variable ‘i’ [-Werror=unused-variable]
1980 | unsigned int i;
| ^
oml.c:1979:35: error: unused variable ‘s4l’ [-Werror=unused-variable]
1979 | const struct lchan_sapis *s4l = &sapis_for_lchan[lchan->type];
| ^~~

oml.c: In function ‘l1if_rsl_chan_mod’:
oml.c:1985:22: warning: unused variable ‘i’ [-Wunused-variable]
1985 | unsigned int i;
| ^
oml.c:1984:35: warning: unused variable ‘s4l’ [-Wunused-variable]
1984 | const struct lchan_sapis *s4l = &sapis_for_lchan[lchan->type];
| ^~~

Change-Id: Ieb8882bbade7f6d6628c9c00150bcf54e3a9dc75
---
M src/osmo-bts-lc15/oml.c
M src/osmo-bts-oc2g/oml.c
2 files changed, 25 insertions(+), 6 deletions(-)

diff --git a/src/osmo-bts-lc15/oml.c b/src/osmo-bts-lc15/oml.c
index 39a9922..9057f9d 100644
--- a/src/osmo-bts-lc15/oml.c
+++ b/src/osmo-bts-lc15/oml.c
@@ -2008,9 +2008,6 @@
*/
int l1if_rsl_chan_mod(struct gsm_lchan *lchan)
{
- const struct lchan_sapis *s4l = &sapis_for_lchan[lchan->type];
- unsigned int i;
-
if (lchan->ho.active == HANDOVER_NONE)
return -1;

diff --git a/src/osmo-bts-oc2g/oml.c b/src/osmo-bts-oc2g/oml.c
index 5dce86f..a8bd3d9 100644
--- a/src/osmo-bts-oc2g/oml.c
+++ b/src/osmo-bts-oc2g/oml.c
@@ -2012,9 +2012,6 @@
*/
int l1if_rsl_chan_mod(struct gsm_lchan *lchan)
{
- const struct lchan_sapis *s4l = &sapis_for_lchan[lchan->type];
- unsigned int i;
-
if (lchan->ho.active == HANDOVER_NONE)
return -1;


To view, visit change 32106. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ieb8882bbade7f6d6628c9c00150bcf54e3a9dc75
Gerrit-Change-Number: 32106
Gerrit-PatchSet: 3
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged