<p>Richard Mudgett <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7761">View Change</a></p><p>Patch set 1:</p><p>(1 comment)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/7761/1/main/stasis_channels.c">File main/stasis_channels.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7761/1/main/stasis_channels.c@175">Patch Set #1, Line 175:</a> <code style="font-family:monospace,monospace">static int channel_snapshot_hash_cb(const void *obj, const int flags)</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">We need case insensitive variants of the AO2_STRING_FIELD_* macros so they </blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Well, generic macros could be created that parameterize the hash and compare functions.  Then the existing macros could call the more generic macros.  Something like that should be done in its own review though rather than creeping the scope of reviews like this.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/7761">change 7761</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/7761"/><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: I02c405518cab22aa2a082b61e2353bf7cd629a70 </div>
<div style="display:none"> Gerrit-Change-Number: 7761 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Richard Mudgett <rmudgett@digium.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: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 28 Dec 2017 21:48:54 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>