<p>Corey Farrell <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7774">View Change</a></p><p>Patch set 1:</p><p>(2 comments)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/7774/1/include/asterisk/astobj2.h">File include/asterisk/astobj2.h:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7774/1/include/asterisk/astobj2.h@2084">Patch Set #1, Line 2084:</a> <code style="font-family:monospace,monospace">        const char *right_key = arg; \</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">The macros assume that the key is a char string.  It doesn't have to be.  T</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">This is why I added the doxygen \warning stating that this macro API could change.  If we need something else in the future we can decide if this macro should become more complex or if the 'something else' should just have it's own macro.</p></li><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7774/1/include/asterisk/astobj2.h@2093">Patch Set #1, Line 2093:</a> <code style="font-family:monospace,monospace">         cmp = partial_key_cmp(object_left->field, right_key, strlen(right_key)); \</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">A partial key search could pass in a different type to avoid doing a strlen</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think this goes with the first comment - do we want to add the complexity needed to support this now?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/7774">change 7774</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/7774"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I11af8c6a0c43380a42732553f519c667abb842cf </div>
<div style="display:none"> Gerrit-Change-Number: 7774 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 02 Jan 2018 19:37:59 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>