<p>Pau Espin Pedrol has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/13664">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mgcp-client: whitespace cleanup in mgcp_client.c<br><br>Change-Id: Ic3495d70cb9c4e12552c6d97481cc0cf04b79f94<br>---<br>M src/libosmo-mgcp-client/mgcp_client.c<br>1 file 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/64/13664/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/libosmo-mgcp-client/mgcp_client.c b/src/libosmo-mgcp-client/mgcp_client.c</span><br><span>index 88e5dab..17f40b9 100644</span><br><span>--- a/src/libosmo-mgcp-client/mgcp_client.c</span><br><span>+++ b/src/libosmo-mgcp-client/mgcp_client.c</span><br><span>@@ -323,8 +323,8 @@</span><br><span>      unsigned int pt;</span><br><span>     char codec_resp[64];</span><br><span>         unsigned int codec;</span><br><span style="color: hsl(0, 100%, 40%);">-     </span><br><span style="color: hsl(0, 100%, 40%);">-        </span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>         if (strstr(line, "ptime")) {</span><br><span>               if (sscanf(line, "a=ptime:%u", &r->ptime) != 1)</span><br><span>                     goto response_parse_failure_ptime;</span><br><span>@@ -353,17 +353,17 @@</span><br><span>           } else</span><br><span>                       goto response_parse_failure_rtpmap;</span><br><span>  }</span><br><span style="color: hsl(0, 100%, 40%);">-       </span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>   return 0;</span><br><span> </span><br><span> response_parse_failure_ptime:</span><br><span>       LOGP(DLMGCP, LOGL_ERROR,</span><br><span>          "Failed to parse SDP parameter, invalid ptime (%s)\n", line);</span><br><span style="color: hsl(0, 100%, 40%);">-    return -EINVAL;         </span><br><span style="color: hsl(120, 100%, 40%);">+      return -EINVAL;</span><br><span> response_parse_failure_rtpmap:</span><br><span>    LOGP(DLMGCP, LOGL_ERROR,</span><br><span>          "Failed to parse SDP parameter, invalid rtpmap (%s)\n", line);</span><br><span style="color: hsl(0, 100%, 40%);">-   return -EINVAL;         </span><br><span style="color: hsl(120, 100%, 40%);">+      return -EINVAL;</span><br><span> }</span><br><span> </span><br><span> /* Parse a line like "c=IN IP4 10.11.12.13" */</span><br><span>@@ -533,7 +533,7 @@</span><br><span>   /* If there is an SDP body attached, prevent for_each_non_empty_line()</span><br><span>        * into running in there, we are not yet interested in the parameters</span><br><span>         * stored there. */</span><br><span style="color: hsl(0, 100%, 40%);">-     data_end = mgcp_find_section_end(data); </span><br><span style="color: hsl(120, 100%, 40%);">+      data_end = mgcp_find_section_end(data);</span><br><span>      if (data_end)</span><br><span>                *data_end = '\0';</span><br><span> </span><br><span>@@ -979,7 +979,7 @@</span><br><span>          for (i = 0; i < mgcp_msg->codecs_len; i++) {</span><br><span>                   pt = mgcp_msg->codecs[i];</span><br><span>                         codec = get_value_string_or_null(osmo_mgcpc_codec_names, pt);</span><br><span style="color: hsl(0, 100%, 40%);">-                   </span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>                   /* Note: Use codec descriptors from enum mgcp_codecs</span><br><span>                          * in mgcp_client only! */</span><br><span>                   OSMO_ASSERT(codec);</span><br><span>@@ -1067,7 +1067,7 @@</span><br><span> </span><br><span>      for (i = 0; i < mgcp_msg->codecs_len; i++) {</span><br><span>           pt = map_codec_to_pt(mgcp_msg->ptmap, mgcp_msg->ptmap_len, mgcp_msg->codecs[i]);</span><br><span style="color: hsl(0, 100%, 40%);">-               </span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>           /* Note: Only dynamic payload type from the range 96-127</span><br><span>              * require to be explained further via rtpmap. All others</span><br><span>             * are implcitly definedby the number in m=audio */</span><br><span>@@ -1077,11 +1077,11 @@</span><br><span>                        /* Note: Use codec descriptors from enum mgcp_codecs</span><br><span>                          * in mgcp_client only! */</span><br><span>                   OSMO_ASSERT(codec);</span><br><span style="color: hsl(0, 100%, 40%);">-                     </span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>                   rc += msgb_printf(msg, "a=rtpmap:%u %s\r\n", pt, codec);</span><br><span>           }</span><br><span>    }</span><br><span style="color: hsl(0, 100%, 40%);">-       </span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>   if (mgcp_msg->ptime)</span><br><span>              rc += msgb_printf(msg, "a=ptime:%u\r\n", mgcp_msg->ptime);</span><br><span> </span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/13664">change 13664</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/13664"/><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-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: Ic3495d70cb9c4e12552c6d97481cc0cf04b79f94 </div>
<div style="display:none"> Gerrit-Change-Number: 13664 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Pau Espin Pedrol <pespin@sysmocom.de> </div>