<p>Alexander Traud <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/8854">View Change</a></p><p>Patch set 7:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">(1 comment)</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Still crashes. However, as you noted it is not reproducible. Now, I get a crash in test/apps/sendtext/enhanced.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Unicode and US-ASCII do not work as encoding because 155 is not defined in those character sets. But that cannot be the cause alone, because I had an error about another codepoint before that test. I cannot help, because I do not understand that source code. On the first glance, looks like you are dealing not with strings but bytes. If you deal with bytes, you can forget any charset in Python because it is one of the languages which does codepage checks = if the codepoint is not defined it bails. US-ASCII is a sub-set of UTF-8. US-ASCII is a subset of ISO-8859-1. ISO-8859-1 is a subset of winlatin1. However, ISO-8859-1 is not a subset of UTF-8. Furthermore, Unicode defines a lot of character sequences as invalid. Therefore and because Python checks, you have to be absolutely sure what you deal with (bytes, characters and which encoding). Otherwise, you can forget it. If you deal with a Terminal as middle-man, … then it depends even on the locale of the user.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Furthermore, I cannot help to test anymore because the whole suite crashes and not simply a test fails. Finally, I see a lot of regressions already until I reach a crash. I am sorry to give no positive result because you did a tremendous work already.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/8854">change 8854</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/8854"/><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: If76c2d3e11e4ab4552d0df7841287c8bb2de7918 </div>
<div style="display:none"> Gerrit-Change-Number: 8854 </div>
<div style="display:none"> Gerrit-PatchSet: 7 </div>
<div style="display:none"> Gerrit-Owner: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 21 Jun 2018 11:07:10 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>