laforge submitted this change.

View Change

Approvals: laforge: Looks good to me, approved pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified
Cosmetic: library/MGCP_CodecPort: remove FIXME

The line below is exactly as it should be according to the FIXME, so
remove it.

Change-Id: If14438edce4437cd30efc3b2520aa43b90e9e16b
---
M library/MGCP_CodecPort.ttcn
1 file changed, 0 insertions(+), 4 deletions(-)

diff --git a/library/MGCP_CodecPort.ttcn b/library/MGCP_CodecPort.ttcn
index 8614eef..8819fd8 100644
--- a/library/MGCP_CodecPort.ttcn
+++ b/library/MGCP_CodecPort.ttcn
@@ -74,10 +74,6 @@
pout.remPort := pin.remPort;
pout.locName := pin.locName;
pout.locPort := pin.locPort;
- /* FIXME: This should actually be the below:
- pout.msg := dec_MgcpMessage(oct2char(pin.msg)); - see
- https://www.eclipse.org/forums/index.php/t/1088893/
- */
pout.msg := dec_MgcpMessage(oct2char(pin.msg));
} with { extension "prototype(fast)" };


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

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: If14438edce4437cd30efc3b2520aa43b90e9e16b
Gerrit-Change-Number: 31335
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged