Attention is currently required from: laforge.
pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-bsc/+/38087?usp=email )
Change subject: Introduce hashtable to lookup bts by LAC ......................................................................
Patch Set 2:
(1 comment)
File src/osmo-bsc/neighbor_ident.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/38087/comment/e0dd77a6_e2d31952?usp=... : PS2, Line 381: f (bts_tmp->location_area_code != lac) : continue;
shouldn't this be now an error message in all of those iterations (not just this function)? After a […]
Not really. AFAIU different LACs can end up in the same hashtable bucket/llist due to hashing function, it's totally fine and expected...
-
pespin