Hi Howard, On Tue, Jul 27, 2021 at 7:51 PM Yun-hao Chung <howardchung@xxxxxxxxxx> wrote: > > > > > if (svc_start < foreach_data->start || > > > > svc_start > foreach_data->end) > > > > return; > After discussing with Archie, if we understand you correctly, we think > this early return can be removed. > Let's say the searched range is 5-9 and the service is 1-14, then we > should consider the service is in range. > If we want to keep the early return, svc_start > foreach_data->end is > already checked a few lines before, so this check is redundant. Yep, that seems we have already had the same check so Im fine dropping the check. -- Luiz Augusto von Dentz
