<p><a href="https://gerrit.asterisk.org/10977">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.asterisk.org/#/c/10977/4/main/strings.c">File main/strings.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/10977/4/main/strings.c@252">Patch Set #4, Line 252:</a> <code style="font-family:monospace,monospace">      if (!str1 || !str2) {</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Should (!str1 && !str2) return 0?  Two NULL's are equal..</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">two NULL's are equal yes, but for this function NULL's are not considered matching strings, or strings at all.</p><p style="white-space: pre-wrap; word-wrap: break-word;">This function is to be used in those situations where you want to compare two strings for equality, but need to first make sure one or the other, or both are not NULL otherwise strcmp would be undefined.</p><p style="white-space: pre-wrap; word-wrap: break-word;">If you feel this has limited use, or is more confusing than worthwhile then I can remove the function altogether. The use case may be less often than I imagined.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/10977">change 10977</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/10977"/><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: I26f0a16d61537505eb41b4b05ef2e6d67fc2541b </div>
<div style="display:none"> Gerrit-Change-Number: 10977 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Kevin Harwell <kharwell@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: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 19 Feb 2019 17:52:48 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>