Change in osmo-bsc[master]: LCLS: use enum values instead of magic numbers

[Max]

Max has uploaded this change for review. ( https://gerrit.osmocom.org/12378


Change subject: LCLS: use enum values instead of magic numbers
......................................................................

LCLS: use enum values instead of magic numbers

Change-Id: I3f49f74edb5400df1b13bb75da3d524f234c8d03
Related: OS#3659
---
M src/osmo-bsc/osmo_bsc_lcls.c
1 file changed, 5 insertions(+), 5 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/78/12378/1

diff --git a/src/osmo-bsc/osmo_bsc_lcls.c b/src/osmo-bsc/osmo_bsc_lcls.c
index 26b3244..2d5a077 100644
--- a/src/osmo-bsc/osmo_bsc_lcls.c
+++ b/src/osmo-bsc/osmo_bsc_lcls.c
@@ -50,11 +50,11 @@
 enum gsm0808_lcls_status lcls_get_status(struct gsm_subscriber_connection 
*conn)
 {
        if (!conn->lcls.fi)
-               return 0xff;
+               return GSM0808_LCLS_STS_NA;

        switch (conn->lcls.fi->state) {
        case ST_NO_LCLS:
-               return 0xff;
+               return GSM0808_LCLS_STS_NA;
        case ST_NOT_YET_LS:
                return GSM0808_LCLS_STS_NOT_YET_LS;
        case ST_NOT_POSSIBLE_LS:
@@ -76,7 +76,7 @@
        enum gsm0808_lcls_status status = lcls_get_status(conn);
        struct msgb *msg;

-       if (status == 0xff)
+       if (status == GSM0808_LCLS_STS_NA)
                return;

        LOGPFSM(conn->lcls.fi, "Sending BSSMAP LCLS NOTIFICATION (%s)\n",
@@ -384,7 +384,7 @@
                        return;
                return;
        case LCLS_EV_APPLY_CFG_CSC:
-               if (conn->lcls.config == 0xff)
+               if (conn->lcls.config == GSM0808_LCLS_CFG_NA)
                        return;
                if (lcls_perform_correlation(conn) != 0) {
                        /* Correlation leads to no result: Not Possible to LS */
@@ -887,7 +887,7 @@
 void bssmap_add_lcls_status_if_needed(struct gsm_subscriber_connection *conn, 
struct msgb *msg)
 {
        enum gsm0808_lcls_status status = lcls_get_status(conn);
-       if (status != 0xff) {
+       if (status != GSM0808_LCLS_STS_NA) {
                LOGPFSM(conn->fi, "Adding LCLS BSS-Status (%s) to %s\n",
                        gsm0808_lcls_status_name(status),
                        gsm0808_bssmap_name(msg->l3h[2]));

--
To view, visit https://gerrit.osmocom.org/12378
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3f49f74edb5400df1b13bb75da3d524f234c8d03
Gerrit-Change-Number: 12378
Gerrit-PatchSet: 1
Gerrit-Owner: Max <msur...@sysmocom.de>