<p style="white-space: pre-wrap; word-wrap: break-word;">I still see some of my points not addressed, but it's better to have this now than nothing. If you will, you could address below review and ping me to +2 again; or merge and come back with anoter patch later; or merge and disagree with me and I will eternally dislike it but probably forget about it soon :P</p><p>Patch set 4:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4;">Code-Review +2</span></p><p><a href="https://gerrit.osmocom.org/11836">View Change</a></p><p>5 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11836/4/common/chapters/gsup.adoc">File common/chapters/gsup.adoc:</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/11836/4/common/chapters/gsup.adoc@509">Patch Set #4, Line 509:</a> <code style="font-family:monospace,monospace">Direction: MSC / SGSN => SMSC (through HLR)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">(a nicer term is "via", not "through" ... but nm)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11836/4/common/chapters/gsup.adoc@541">Patch Set #4, Line 541:</a> <code style="font-family:monospace,monospace">_MO (Mobile Originated)_ short message delivery. The corresponding</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I still think it is really irritating to explain the same abbreviations over and over. If you have to, place the braces once above and then don't do that anymore for the rest of the doc. But actually, instead make sure MO and MT are in the glossary and then rely on that completely. We did agree on that before, right? I think we also agreed before on less fancy fontsy stuff, i.e. try to avoid italics and bold script. Especially don't place an entire "FOO (yada)" in italics?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11836/4/common/chapters/gsup.adoc@543">Patch Set #4, Line 543:</a> <code style="font-family:monospace,monospace">section 12.2.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">it's much nicer with the reference in the end, thanks :)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11836/4/common/chapters/gsup.adoc@578">Patch Set #4, Line 578:</a> <code style="font-family:monospace,monospace">This message is used to forward _MT (Mobile Terminated)_ short messages</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">MT from glossary</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11836/4/common/chapters/gsup.adoc@579">Patch Set #4, Line 579:</a> <code style="font-family:monospace,monospace">from an SMSC to MSC (CS domain) / SGSN (PS domain). The corresponding MAP</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">and I still don't like this. I am still of the same opinion: the protocol definition is not the place to explain MSC = CS domain and SGSN = PS domain, especially not N times over and over.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11836">change 11836</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/11836"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-gsm-manuals </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ie0150756c33c1352bc4eb49421824542c711175c </div>
<div style="display:none"> Gerrit-Change-Number: 11836 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Vadim Yanitskiy <axilirator@gmail.com> </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: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 10 Dec 2018 20:46:25 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>