laforge submitted this change.

View Change


Approvals: fixeria: Looks good to me, approved pespin: Looks good to me, approved Jenkins Builder: Verified
don't log useless "transcoding disabled" message

The entire mgw has no transcoding support. So printing that message is
useless to begin with. And printing it for *every RTP packet* is even
more useless. Let's remove it.

Change-Id: If0ee2607404afc3a00665a5cf22a9e0eb62eb476
---
M src/libosmo-mgcp/mgcp_network.c
1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/src/libosmo-mgcp/mgcp_network.c b/src/libosmo-mgcp/mgcp_network.c
index 111c5cf..ae0a52d 100644
--- a/src/libosmo-mgcp/mgcp_network.c
+++ b/src/libosmo-mgcp/mgcp_network.c
@@ -412,7 +412,6 @@
struct mgcp_rtp_end *dst_end,
char *data, int *len, int buf_size)
{
- LOGPENDP(endp, DRTP, LOGL_DEBUG, "transcoding disabled\n");
return 0;
}


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

Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Change-Id: If0ee2607404afc3a00665a5cf22a9e0eb62eb476
Gerrit-Change-Number: 36358
Gerrit-PatchSet: 2
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged