<p style="white-space: pre-wrap; word-wrap: break-word;">I left the default value for SMSC as is because I don't know why we chose it and I don't think there is some sane default that we could default to. It would be nice for pySim-prog to not program everything, but only the values that are specified, but that is a patch/discussion for another time.</p><p><a href="https://gerrit.osmocom.org/11201">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11201/1/pySim-prog.py">File pySim-prog.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/11201/1/pySim-prog.py@100">Patch Set #1, Line 100:</a> <code style="font-family:monospace,monospace">                 help="SMSC number (Start with + for international no.) [default: '00 + country code + 5555']",</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">"If it starts with a '+', Type-of-Number is programmed as 'international'" ? […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11201/1/pySim-prog.py@325">Patch Set #1, Line 325:</a> <code style="font-family:monospace,monospace">                     if smsc[0] == '+':</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">cosmetic: the rest of the file is indented by tabs, the new lines by spaces</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11201/1/pySim-prog.py@329">Patch Set #1, Line 329:</a> <code style="font-family:monospace,monospace">                            raise ValueError('SMSC must be digits only!\n \</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">"and optionally start with a '+'" ?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11201">change 11201</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/11201"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ia79913f5b0307e9786a5acea75c0811927be2eef </div>
<div style="display:none"> Gerrit-Change-Number: 11201 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 05 Oct 2018 11:22:12 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>