Attention is currently required from: osmith.

View Change

2 comments:

• File src/host/layer23/src/common/apn_fsm.c:

  • Patch Set #2, Line 103: apn_fsm_state_chg(fi, APN_ST_DISABLED);

    Don't we need to check *((bool *)data)? I guess the assumption is that it is false here, as I unders […]

    In theory it's not really needed, since in this state we know for sure the current state is to have GPRS enabled, so I'd expect the event to be "disable" here, but yeah, let's test it to make sure.

  • Patch Set #2, Line 139: apn_fsm_state_chg(fi, APN_ST_INACTIVE);

    * is this correct, receiving CTX_ACC and then going to inactive here? […]

    Yes, because it's "DEACT".
    I left out REJ for now, we are not even implementing most of deactivation anyway so far, and I need to figure out how to trigger app-specific code to instruct GMM/SM layers to do stuff from the common/ code.

    So yes, this FSM will probably be extended in the future. The problem is that I can't simply implement everything at the same time in one commit, that would be impossible to review. Pushing this in small bits allows other people already testing and contributing too.

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