<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;"><br>My concern is that this patch is completely disabling inlining for<br>some tiny functions.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Someone had asked why we couldn't just remove the `force_inline`<br>but leave these in the headers, it was pointed out that GCC would<br>complain about the functions being unused.  Couldn't we just<br>replace `force_inline` with `attribute_unused`?  This way GCC won't<br>complain about the unused static functions and we'd still allow it<br>to be inlined if GCC wants.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Hi Corey, maybe there is a bit of a confusion, there is one comment back there I entered wrong and couldn't delete and tried to rephrased it in the subsequent comment. In any case here is to clarfy: This patch does not use flag -fno-partial-inline. It was discussed that since this is a master version it will be okay to move the funcions out of the header file since master versions don't mind about ABI breakages. Therefore it will be okay with removing the force_inline from the prototypes in include/asterisk/strings.h.  Not sure if that was your concern.<br>Thanks</p><p><a href="https://gerrit.asterisk.org/10936">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/10936">change 10936</a>. To unsubscribe, or for help writing mail filters, 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/10936"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I2a90249763a7139fd12c54ad4e0e8d5cbbed14f9 </div>
<div style="display:none"> Gerrit-Change-Number: 10936 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Chris Savinovich <csavinovich@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Chris Savinovich <csavinovich@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation (1000185) </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Joshua C. Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Matthew Fredrickson <creslin@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Walter Doekes <walter+asterisk@wjd.nu> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 18 Feb 2019 18:14:49 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>