Attention is currently required from: jolly.

View Change

5 comments:

• File s1gw/S1GW_ConnHdlr.ttcn:

  • Patch Set #2, Line 1706:

    	var ERabIdxList erabs_admitted;
    	for (var integer i := 0; i < lengthof(g_pars.erabs); i := i + 1) {
    		erabs_admitted[i] := i;
    	}
    

    See my comment about `f_ERabList_compose()` in previous patch.

  • Patch Set #2, Line 1720:

    f_ConnHdlr_tx_handover_ack(erabs_admitted, mme_ue_id, enb_ue_id, erabs_failed);
    	log("[eNB -> S1GW] -> MME: HANDOVER REQUEST ACKNOWLEDGE");
    

    We usually log first:

    ```suggestion
    	log("[eNB -> S1GW] -> MME: HANDOVER REQUEST ACKNOWLEDGE");
    	f_ConnHdlr_tx_handover_ack(erabs_admitted, mme_ue_id, enb_ue_id, erabs_failed);
    ```

  • Patch Set #2, Line 1726:

    for (var integer i := 0; i < lengthof(erabs_admitted); i := i + 1) {
    			var ERabIdx idx := erabs_admitted[i];
    			erabs[i] := g_pars.erabs[idx];
    		}
    

    See my comment about `f_ERabList_compose()` in previous patch.

  • Patch Set #2, Line 1736:

    		for (var integer i := 0; i < lengthof(erabs_failed); i := i + 1) {
    			var ERabIdx idx := erabs_failed[i];
    			erabs[i] := g_pars.erabs[idx];
    		
    

    See my comment about `f_ERabList_compose()` in previous patch.

• File s1gw/S1GW_Tests.ttcn:

  • Patch Set #2, Line 962:

                    for (var integer i := 0; i < lengthof(erabs_admitted); i := i + 1) {
                            var ERabIdx idx := erabs_admitted[i];
                            erabs_left[i] := g_pars.erabs[idx];
      
    

    See my comment about `f_ERabList_compose()` in previous patch.

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