neels has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bsc/+/19559 )
Change subject: hodec2: do not keep candidates with zero requirements met ...................................................................... hodec2: do not keep candidates with zero requirements met When check_requirements() returns zero, do not keep such an entry in the candidates list at all. This removes logging confusion, where some "candidates" are still listed even though not meeting any handover requirements. Change-Id: I12e48292d5731cb601165c870b9570003bc488ec --- M src/osmo-bsc/handover_decision_2.c 1 file changed, 7 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/59/19559/1 diff --git a/src/osmo-bsc/handover_decision_2.c b/src/osmo-bsc/handover_decision_2.c index 26a8840..3191957 100644 --- a/src/osmo-bsc/handover_decision_2.c +++ b/src/osmo-bsc/handover_decision_2.c @@ -861,6 +861,10 @@ }; debug_candidate(&c, 0, tchf_count, tchh_count); + + if (!c.requirements) + return; + clist[*candidates] = c; (*candidates)++; } @@ -966,6 +970,9 @@ debug_candidate(&c, av_rxlev, tchf_count, tchh_count); + if (!c.requirements) + return; + clist[*candidates] = c; (*candidates)++; } -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/19559 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I12e48292d5731cb601165c870b9570003bc488ec Gerrit-Change-Number: 19559 Gerrit-PatchSet: 1 Gerrit-Owner: neels <nhofm...@sysmocom.de> Gerrit-MessageType: newchange