Attention is currently required from: fixeria, msuraev.

View Change

2 comments:

• File src/host/layer23/src/mobile/app_mobile.c:

  • Patch Set #1, Line 249: ms->mncc_entity.sock_state = mncc_sock_init(ms, ms->settings.mncc_socket_path);

    So sock_state is used only if mncc_handler is "EXTERNAL"? Then better put it under ms->mncc_entity.external.sock_state

• File src/host/layer23/src/mobile/vty_interface.c:

  • Patch Set #1, Line 1665: DEFUN(cfg_ms_no_mncc_handler, cfg_ms_no_mncc_handler_cmd,

    What's the point of introducing redundant vty command if dummy handler can be selected with mncc-han […]

    ACK, specifying the default above in the description (like you did) should be enough for the user to set the mncc-handler to default.

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