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 <pespin(a)sysmocom.de> Gerrit-MessageType: newchange