<p><a href="https://gerrit.osmocom.org/c/pysim/+/23803">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/pysim/+/23803/1/pySim-shell.py">File pySim-shell.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/pysim/+/23803/1/pySim-shell.py@293">Patch Set #1, Line 293:</a> <code style="font-family:monospace,monospace">           headli</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">so we are appending 80 spaces....<br>only to then cut out only a chunk of 77 characters from that one line below?  I find that somewhat odd, and cannot really undersatnd why.  Why are you not formatting the whole line in one? if for some reason that's not possible with string concatenation, simply use the '%' operator like '# %77s #' % ('Export summary')</p><p style="white-space: pre-wrap; word-wrap: break-word;">And actually, I would move all of that to a function "print a heading with a surrounding box".  Or, as we are using poutput(), maybe rathe "create a string containing a boxed heading" so one can simply say</p><p style="white-space: pre-wrap; word-wrap: break-word;">self._cmd.poutput(format_as_box('Export summary')).</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/pysim/+/23803">change 23803</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/c/pysim/+/23803"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: pysim </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I90771e525b2b114bdb41a8e90d298ca991c09c3d </div>
<div style="display:none"> Gerrit-Change-Number: 23803 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 21 Apr 2021 06:44:59 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>