msuraev has submitted this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/30207 ) Change subject: cosmetic: drop duplicated comment ...................................................................... cosmetic: drop duplicated comment Change-Id: Id349fbfe683b7b63ac9076d2e9998f7c3ecea112 --- M src/pdch_ul_controller.c 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins Builder: Verified osmith: Looks good to me, but someone else must approve pespin: Looks good to me, approved diff --git a/src/pdch_ul_controller.c b/src/pdch_ul_controller.c index df0c500..9cbe845 100644 --- a/src/pdch_ul_controller.c +++ b/src/pdch_ul_controller.c @@ -171,7 +171,7 @@ /* Next reserved FN is passed check_fn, hence it means check_fn is free */ return check_fn; } - /* if it->fn < check_fn, simply continue iterating, we want to reach at least check_fn */ + if (res == 0)/* it->fn == fn */ check_fn = fn_next_block(check_fn); /* if it->fn < check_fn, simply continue iterating, we want to reach at least check_fn */ -- To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/30207 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-pcu Gerrit-Branch: master Gerrit-Change-Id: Id349fbfe683b7b63ac9076d2e9998f7c3ecea112 Gerrit-Change-Number: 30207 Gerrit-PatchSet: 2 Gerrit-Owner: msuraev &lt;msuraev(a)sysmocom.de&gt; Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria &lt;vyanitskiy(a)sysmocom.de&gt; Gerrit-Reviewer: msuraev &lt;msuraev(a)sysmocom.de&gt; Gerrit-Reviewer: osmith &lt;osmith(a)sysmocom.de&gt; Gerrit-Reviewer: pespin &lt;pespin(a)sysmocom.de&gt; Gerrit-MessageType: merged