Attention is currently required from: fixeria, laforge.

Patch set 1:Code-Review +1

View Change

2 comments:

• Patchset:

  • Patch Set #1:

    i think it's cool for lab devel, removing one layer of indirection of keeping track which phone has what uSIM.

    (Also reminds me of reports where Keith told us about how hard it can be in practice to find out the actual IMSI of a device someone holds in your face. So in that particular context it seems silly to use the IMSI.)

    It's so simple, I'm +1

• File include/osmocom/hlr/hlr.h:

  • Patch Set #1, Line 46: SUBSCR_COD_MODE_IMSI,

    just "mode IMSI" wasn't clear to me until I read the code below that copies the IMSI to the MSISDN. Maybe more specific naming based on "mode IMSI-is-MSISDN" or "mode MSISDN-from-IMSI"?

    (The other mode names have "MSISDN" in the name, so at first reading I thought "what, a randomly generated IMSI??")

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