Attention is currently required from: dexter.

View Change

3 comments:

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

  • Patch Set #2, Line 121: struct mgcp_msg {

    If this struct is being allocated outside of the library, now it's for sure the moment to add an alloc function! :)

• File include/osmocom/mgcp_client/mgcp_client_fsm.h:

  • Patch Set #2, Line 70: struct mgcp_codec_param2 codecs_param[MGCP_MAX_CODECS];

    as soon as you add a field to "struct mgcp_codec_param2", the size of the struct will and change any program using the old size will really be fucked up doing weird stuff on this array.
    This should be given a second thought in order to improve the API so that we don't go breaking stuff every time we want to support something new.

    
    We should start by having an API to allocate this struct from within the library...

• File src/libosmo-mgcp-client/mgcp_client.c:

  • Patch Set #2, Line 1376: LOGPMGW(mgcp, LOGL_DEBUG, "using depreacted struct member mgcp_msg->param, please use mgcp_msg->codecs_param!\n");

    only debug?

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