<p dir="ltr"><br>
18 февр. 2014 г. 11:02 пользователь "Holger Hans Peter Freyther" <<a href="mailto:holger@freyther.de">holger@freyther.de</a>> написал:<br>
><br>
> On Mon, Feb 17, 2014 at 04:50:42PM +0400, Ivan Kluchnikov wrote:<br>
><br>
> Good Morning,<br>
><br>
> > +             llist_for_each_entry(cur_trx, &trx->bts->trx_list, list) {<br>
> > +                     if (cur_trx->arfcn == arfcn) {<br>
><br>
> I wondered if we want to compare the access reference as well? We do not<br>
> enforce that the ARFCNs of a TRX are distinct.</p>
<p dir="ltr">Actually I think we should rather start enforcing that. Or do you know situations when both TRX's could run on the same ARFCN?</p>
<p dir="ltr">> And besides that I would<br>
> like you to move the look-up code to a separate function. That way we can<br>
> at least write a unit test in the future.</p>
<p dir="ltr">+1<br></p>
<p dir="ltr">Please excuse typos. Written with a touchscreen keyboard.</p>
<p dir="ltr">--<br>
Regards,<br>
Alexander Chemeris<br>
CEO/Founder Fairwaves LLC<br>
<a href="http://fairwaves.ru">http://fairwaves.ru</a></p>