<p>Corey Farrell <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/8918">View Change</a></p><p>Patch set 1:</p><p style="white-space: pre-wrap; word-wrap: break-word;">Revision 1 of this review is posted from the github.com pull request.  I only rebased to current master and amended the commit message to have the Change-Id added.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Also posted comment from github to get comments from gerrit reviewers.</p><p>(1 comment)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/8918/1/starpy/manager.py">File starpy/manager.py:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/8918/1/starpy/manager.py@122">Patch Set #1, Line 122:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">        types = [str, type(None)]<br>        if sys.version_info <= (3, 0):<br>            types.append(unicode)<br>        if isinstance(event, tuple(types)):<br>            event = (event,)<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">This (and the second copy of the same code) could either:</p><ul><li>Use six.string_types</li><li>Use a locally implemented function of the same</li><li>Left alone.</li></ul><p style="white-space: pre-wrap; word-wrap: break-word;">Personally I prefer a local function, I don't think this one feature is worth adding an external dependency.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/8918">change 8918</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/8918"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: starpy </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I9ccd5a15be6e4129a5e0ce0bf1a5a4ff26998bac </div>
<div style="display:none"> Gerrit-Change-Number: 8918 </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: Rodrigo Ramirez Norambuena <a@rodrigoramirez.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 03 May 2018 13:01:45 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>