<p style="white-space: pre-wrap; word-wrap: break-word;">Please elaborate - either complete function as you'd like to see it or test-case which you think current version would fail. I have hard time wrapping my head around this.</p><p><a href="https://gerrit.osmocom.org/11929">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11929/2/osmopy/osmo_ipa.py">File osmopy/osmo_ipa.py:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11929/2/osmopy/osmo_ipa.py@126">Patch Set #2, Line 126:</a> <code style="font-family:monospace,monospace">        if (int(length) != 0):</code></p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">if length == 0 or payload...</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Either I'm not following or this doesn't make sense. What's the point of go into recursion on 2nd message if the length of the 1st one already indicates that we should return None?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11929">change 11929</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.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.osmocom.org/11929"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: python/osmo-python-tests </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I51ce207c19a1ca96c3e2af7d5efd64f79b02fbb4 </div>
<div style="display:none"> Gerrit-Change-Number: 11929 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 27 Nov 2018 11:00:31 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>