<p>dexter has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-mgw/+/18537">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mgcp: find better locations for LOGPCONN and LOGPENDP<br><br>The logging defines LOGPCONN and LOGPENDP are currently located in<br>mgcp_internal.h. However, there are specific header files for conn<br>(mgcp_conn.h) and endpoint (mgcp_endp.h) related stuff. Lets put LOGPCON<br>into mgcp_conn.h and LOGPENDP in mgcp_endp.h<br><br>Change-Id: I25ff37ee8108c27d169d294fd16ddcdde9b00195<br>---<br>M include/osmocom/mgcp/mgcp_conn.h<br>M include/osmocom/mgcp/mgcp_endp.h<br>M include/osmocom/mgcp/mgcp_internal.h<br>3 files changed, 10 insertions(+), 10 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-mgw refs/changes/37/18537/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/include/osmocom/mgcp/mgcp_conn.h b/include/osmocom/mgcp/mgcp_conn.h</span><br><span>index b0978a6..f4e89f2 100644</span><br><span>--- a/include/osmocom/mgcp/mgcp_conn.h</span><br><span>+++ b/include/osmocom/mgcp/mgcp_conn.h</span><br><span>@@ -28,6 +28,11 @@</span><br><span> #include <osmocom/core/rate_ctr.h></span><br><span> #include <inttypes.h></span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+#define LOGPCONN(conn, cat, level, fmt, args...) \</span><br><span style="color: hsl(120, 100%, 40%);">+LOGPENDP((conn)->endp, cat, level, "CI:%s " fmt, \</span><br><span style="color: hsl(120, 100%, 40%);">+         (conn)->id, \</span><br><span style="color: hsl(120, 100%, 40%);">+         ## args)</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> /* RTP connection related counters */</span><br><span> enum {</span><br><span>  IN_STREAM_ERR_TSTMP_CTR,</span><br><span>diff --git a/include/osmocom/mgcp/mgcp_endp.h b/include/osmocom/mgcp/mgcp_endp.h</span><br><span>index 75f093d..4c2caad 100644</span><br><span>--- a/include/osmocom/mgcp/mgcp_endp.h</span><br><span>+++ b/include/osmocom/mgcp/mgcp_endp.h</span><br><span>@@ -27,6 +27,11 @@</span><br><span> struct mgcp_conn;</span><br><span> struct mgcp_endpoint;</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+#define LOGPENDP(endp, cat, level, fmt, args...) \</span><br><span style="color: hsl(120, 100%, 40%);">+LOGP(cat, level, "endpoint:0x%x " fmt, \</span><br><span style="color: hsl(120, 100%, 40%);">+     endp ? ENDPOINT_NUMBER(endp) : -1, \</span><br><span style="color: hsl(120, 100%, 40%);">+     ## args)</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> /* Callback type for RTP dispatcher functions</span><br><span>    (e.g mgcp_dispatch_rtp_bridge_cb, see below) */</span><br><span> typedef int (*mgcp_dispatch_rtp_cb) (int proto, struct sockaddr_in *addr,</span><br><span>diff --git a/include/osmocom/mgcp/mgcp_internal.h b/include/osmocom/mgcp/mgcp_internal.h</span><br><span>index e9d5d2d..cbf533f 100644</span><br><span>--- a/include/osmocom/mgcp/mgcp_internal.h</span><br><span>+++ b/include/osmocom/mgcp/mgcp_internal.h</span><br><span>@@ -340,13 +340,3 @@</span><br><span> </span><br><span> void mgcp_get_local_addr(char *addr, struct mgcp_conn_rtp *conn);</span><br><span> void mgcp_conn_watchdog_kick(struct mgcp_conn *conn);</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-#define LOGPENDP(endp, cat, level, fmt, args...) \</span><br><span style="color: hsl(0, 100%, 40%);">-LOGP(cat, level, "endpoint:0x%x " fmt, \</span><br><span style="color: hsl(0, 100%, 40%);">-     endp ? ENDPOINT_NUMBER(endp) : -1, \</span><br><span style="color: hsl(0, 100%, 40%);">-     ## args)</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-#define LOGPCONN(conn, cat, level, fmt, args...) \</span><br><span style="color: hsl(0, 100%, 40%);">-LOGPENDP((conn)->endp, cat, level, "CI:%s " fmt, \</span><br><span style="color: hsl(0, 100%, 40%);">-         (conn)->id, \</span><br><span style="color: hsl(0, 100%, 40%);">-         ## args)</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-mgw/+/18537">change 18537</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-mgw/+/18537"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-mgw </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I25ff37ee8108c27d169d294fd16ddcdde9b00195 </div>
<div style="display:none"> Gerrit-Change-Number: 18537 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>