laforge submitted this change.

Approvals:
  osmith: Looks good to me, but someone else must approve
  pespin: Looks good to me, approved
  Jenkins Builder: Verified
  laforge: Looks good to me, approved

gbproxy: Don't log the payload size for every DL/UL-UNITDATA

This just wastes memory. We can check the pcaps if a test does fail.

Change-Id: I862d67a48f1d694262cf39a1e939033bde0c6eab
Related: OS#6704
---
M gbproxy/GBProxy_Tests.ttcn
1 file changed, 0 insertions(+), 3 deletions(-)

diff --git a/gbproxy/GBProxy_Tests.ttcn b/gbproxy/GBProxy_Tests.ttcn
index 7a6572a..7f70f96 100644
--- a/gbproxy/GBProxy_Tests.ttcn
+++ b/gbproxy/GBProxy_Tests.ttcn
@@ -1231,7 +1231,6 @@
 		/* we cannot use pdu_tx as there are some subtle differences in the length field :/ */
 		var template (present) PDU_BSSGP pdu_rx := tr_BSSGP_UL_UD(g_pars.tlli, bvcc.cell_id, payload);
 
-		log("UL-UNITDATA(payload_size=", i);
 		f_pcu2sgsn(pdu_tx, pdu_rx);
 	}
 	setverdict(pass);
@@ -1261,7 +1260,6 @@
 		/* we cannot use pdu_tx as there are some subtle differences in the length field :/ */
 		var template (present) PDU_BSSGP pdu_rx := tr_BSSGP_UL_UD(g_pars.tlli, bvcc.cell_id, payload);
 
-		log("UL-UNITDATA(payload_size=", i);
 		f_pcu2sgsn(pdu_tx, pdu_rx);
 	}
 	setverdict(pass);
@@ -1319,7 +1317,6 @@
 		var template (present) PDU_BSSGP pdu_rx :=
 			tr_BSSGP_DL_UD(g_pars.tlli, payload, tr_BSSGP_IMSI(g_pars.imsi));
 
-		log("DL-UNITDATA(payload_size=", i, ")");
 		f_sgsn2pcu(pdu_tx, pdu_rx);
 	}
 	setverdict(pass);

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