pespin has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-pcu/+/27634 )


Change subject: ms: validate codel enabled against proper define
......................................................................

ms: validate codel enabled against proper define

This is a cosmetic change, since LLC_CODEL_DISABLE is defined as 0.
It clarifies though the meaning of the if condition.

Change-Id: I7c4bda5977a94c2aaf01ca368570f80fd6ebae00
---
M src/gprs_ms.c
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/34/27634/1

diff --git a/src/gprs_ms.c b/src/gprs_ms.c
index c10eda7..3167dee 100644
--- a/src/gprs_ms.c
+++ b/src/gprs_ms.c
@@ -119,7 +119,7 @@
        if (ms->bts)
                codel_interval = the_pcu->vty.llc_codel_interval_msec;

-       if (codel_interval) {
+       if (codel_interval != LLC_CODEL_DISABLE) {
                if (codel_interval == LLC_CODEL_USE_DEFAULT)
                        codel_interval = GPRS_CODEL_SLOW_INTERVAL_MS;
                ms->codel_state = talloc(ms, struct gprs_codel);

--
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/27634
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I7c4bda5977a94c2aaf01ca368570f80fd6ebae00
Gerrit-Change-Number: 27634
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pes...@sysmocom.de>
Gerrit-MessageType: newchange

Reply via email to