laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-mgw/+/36358?usp=email )

Change subject: don't log useless "transcoding disabled" message
......................................................................

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(-)

Approvals:
  fixeria: Looks good to me, approved
  pespin: Looks good to me, approved
  Jenkins Builder: Verified




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 https://gerrit.osmocom.org/c/osmo-mgw/+/36358?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Change-Id: If0ee2607404afc3a00665a5cf22a9e0eb62eb476
Gerrit-Change-Number: 36358
Gerrit-PatchSet: 2
Gerrit-Owner: laforge <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <[email protected]>
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged

Reply via email to