<p>Richard Mudgett <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/8209">View Change</a></p><p>Patch set 4:</p><p>(1 comment)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/8209/4/include/asterisk/mod_format.h">File include/asterisk/mod_format.h:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/8209/4/include/asterisk/mod_format.h@47">Patch Set #4, Line 47:</a> <code style="font-family:monospace,monospace">  char mime_types[80]; /*!< MIME Types related to the format (separated by | if more than one)*/</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Sorry for getting lost on this. I was totally stuck on another issue with A</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">ABI - Application Binary Interface<br>API - Application Programming Interface</p><p style="white-space: pre-wrap; word-wrap: break-word;">Precompiled modules won't know about this new struct member so the existing struct members will be in the wrong place and effectively contain garbage.  Since this struct is normally allocated as a global variable in the format modules you cannot even add the new member to the end of the struct.  The new character array would effectively contain garbage.  This is why this patch can only go in the master branch.  In the master branch you can break interfaces because the code is not released for use yet.</p><p style="white-space: pre-wrap; word-wrap: break-word;">As for repeatedly parsing the string, I don't think the extra CPU needed for parsing the string is much of a problem to consider alternate encodings.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/8209">change 8209</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.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.asterisk.org/8209"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 15 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I1b14692a49b2c1ac67688f58757184122e92ba89 </div>
<div style="display:none"> Gerrit-Change-Number: 8209 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Sean Bright <sean.bright@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Gaurav Khurana <gkhurana@godaddy.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Matt Jordan <mjordan@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Sean Bright <sean.bright@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 27 Feb 2018 14:33:40 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>