[Asterisk-code-review] Begin work on python3 compatability. (testsuite[master])

[Kevin Harwell]

Kevin Harwell has posted comments on this change. ( https://gerrit.asterisk.org/8854 )

Change subject: Begin work on python3 compatability.
......................................................................


Patch Set 9: Code-Review-1

(2 comments)

https://gerrit.asterisk.org/#/c/8854/9/lib/python/asterisk/self_test/test_matcher.py
File lib/python/asterisk/self_test/test_matcher.py:

https://gerrit.asterisk.org/#/c/8854/9/lib/python/asterisk/self_test/test_matcher.py@1
PS9, Line 1: #!/usr/bin/env python
Does test2_matcher still need to exist after the python2/3 compat changes go in?


https://gerrit.asterisk.org/#/c/8854/9/runtests.py
File runtests.py:

https://gerrit.asterisk.org/#/c/8854/9/runtests.py@677
PS9, Line 677:         tbl = {}
             :         for r in bad_chars:
             :             # we do +1 here to include the last item
             :             for i in range(r[0], r[1] + 1):
             :                 tbl[chr(i)] = None
             :         return data.translate(tbl)
I think I am misreading something here. Shouldn't translate take a string as its param and not a dict?



-- 
To view, visit https://gerrit.asterisk.org/8854
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If76c2d3e11e4ab4552d0df7841287c8bb2de7918
Gerrit-Change-Number: 8854
Gerrit-PatchSet: 9
Gerrit-Owner: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Comment-Date: Thu, 05 Jul 2018 16:06:27 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: Yes
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180705/532b88d8/attachment.html>