<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 3:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 2: Code-Review-1</p><p style="white-space: pre-wrap; word-wrap: break-word;">(4 comments)</p><p style="white-space: pre-wrap; word-wrap: break-word;">The DUNDi documentation also needs to be updated with this new functionality.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Apologies for the late response. I have looked and updated based on your review. I have a few comments in regards to your comments.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Line 4976: This isn't a particularly good name for this. This is really a bind address for IPv6, while bindaddr is for IPv4.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Response: bindaddr/bindaddr2 are not restricted to a specific protocol. ie, you could define bindaddr to an IPv6 address and bindaddr2 to an IPv4 address and vice versa. This also allows you to only bind to one address and that be an IPv6 address. If bindaddr inst defined then it is bound to IPv4 on all addresses and bindaddr2 needs to be IPv6 as the bindaddr/2 need to be different protocols. Therefore I don't think this has been named inappropriately. </p><p style="white-space: pre-wrap; word-wrap: break-word;">Line 5195: This should be more explicit about what is needed. You have explicit logic that expects the second one to be IPv6.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Response: See above comment, but again I dont believe there is explicit logic. I have tested the first address being IPv6 and the second address being IPv4. Could you point me to where the logic is restricting?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Looking at the code again I believe you are correct and I was confused originally. It's hard to follow at times.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/10860">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/10860">change 10860</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/c/asterisk/+/10860"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 16 </div>
<div style="display:none"> Gerrit-Change-Id: I5d6e6b52dbe51415046bb3953fd16f5b421bc2e1 </div>
<div style="display:none"> Gerrit-Change-Number: 10860 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Kirsty Tyerman <kirsty.tyerman@boeing.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Kirsty Tyerman <kirsty.tyerman@boeing.com> </div>
<div style="display:none"> Gerrit-Reviewer: mattf <creslin@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 18 Apr 2019 09:17:52 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>