<p>Corey Farrell <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/8691">View Change</a></p><p>Patch set 5:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p>(4 comments)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/8691/5/lib/python/asterisk/utils_socket.py">File lib/python/asterisk/utils_socket.py:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/8691/5/lib/python/asterisk/utils_socket.py@222">Patch Set #5, Line 222:</a> <code style="font-family:monospace,monospace">        empty list.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Is this true?  Seems like we throw an exception if we cannot get ports (or we should).</p></li><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/8691/5/lib/python/asterisk/utils_socket.py@228">Patch Set #5, Line 228:</a> <code style="font-family:monospace,monospace">                return self.get_avail(host, range(port, port + span + step,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This grabs an excess port if span > 0.  If span = 4:<br>range(port, port + 4 + 1, 1) gives a list of 5 ports.</p></li><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/8691/5/lib/python/asterisk/utils_socket.py@233">Patch Set #5, Line 233:</a> <code style="font-family:monospace,monospace">            ports = self.get_avail(host, range(port[0] + 1, port[0] + span +</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think this grabs excess ports.  Example:<br>When we call this with port=0, span=4 would indicate we want a list of 4 ports.  port = self.get_avail(host, 0,...) gets the first port which means we need 3 more.  range(port[0] + 1, port[0] + 4 + 1, 1) gives us 4 more ports but we only want 3 more.</p></li><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/8691/5/lib/python/asterisk/utils_socket.py@238">Patch Set #5, Line 238:</a> <code style="font-family:monospace,monospace">        PortError(socktype, family, attempts)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Do you not need to 'raise PortError(...)'?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/8691">change 8691</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/8691"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: testsuite </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I3da461123afc30e1f5ca12e65d289eaa42d6de00 </div>
<div style="display:none"> Gerrit-Change-Number: 8691 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </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: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 18 Apr 2018 15:52:11 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>