Attention is currently required from: fixeria.

View Change

2 comments:

• File src/osmo-bsc/paging.c:

  • Patch Set #1, Line 92: to_be_deleted->bts

    Why not paging_bts->bts_statg? I see it's used above.

    Because "paging_bts" is not a gsm_bts, but a struct gsm_bts_paging_state, so it would be paging_bts->bts->bts_statg. So it doesn't really matter, it's the same bts pointer in the end.

  • Patch Set #1, Line 470: bts

    This is also confusing: above you're decrementing pending_requests_len via 'bts_entry', but here you […]

    Same, bts_entry is struct gsm_bts_paging_state, which is basically "bts->paging".

To view, visit change 30277. To unsubscribe, or for help writing mail filters, visit settings.