<p style="white-space: pre-wrap; word-wrap: break-word;">Let me state again that this documentation is fine as it is. I have quite detailed opinions on documentation, and it's intended to provide ideas (not so much criticism), and shouldn't be a blocker if others think it's fine.</p><p style="white-space: pre-wrap; word-wrap: break-word;">There are two(?) places where the reference to a section number isn't clear, that's the only reason why I -1.</p><p><a href="https://gerrit.osmocom.org/11836">View Change</a></p><p>7 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11836/2/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/2/common/chapters/gsup.adoc@507">Patch Set #2, Line 507:</a> <code style="font-family:monospace,monospace">==== MO-forwardSM Request</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Well, this mix is not my invention. I am also not a big fun of such style. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">if Wireshark are the only ones using this naming, then that's not a reason to take it in here. If it were etsi specs that would be different.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11836/2/common/chapters/gsup.adoc@509">Patch Set #2, Line 509:</a> <code style="font-family:monospace,monospace">Direction: SGSN / MSC => ESME (through VLR and HLR)</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">It isn't able yet, but there should be an issue about that. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">yes, not important. (yet ever so slightly confusing the status quo)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11836/2/common/chapters/gsup.adoc@522">Patch Set #2, Line 522:</a> <code style="font-family:monospace,monospace">This message is used to forward _MO (Mobile Originated)_ short messages</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Actually, there is some magic happening between "When a subscriber […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">yes, it is absolutely possible to understand, I just found myself reading the sentence was hard with all the interleaves, i.e. the braces like "(PS domain)", "(External...)", "(defined...)", so I would like to have a plain readable sentence first, and then more information separately, if it's required.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11836/2/common/chapters/gsup.adoc@528">Patch Set #2, Line 528:</a> <code style="font-family:monospace,monospace">attempt and _RP-SMMA_ notification attempt on the _SM-RL (Relay Layer)_.</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">> "attempt" sounds like it fails […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">agree (actually none of my remarks are critical)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11836/2/common/chapters/gsup.adoc@532">Patch Set #2, Line 532:</a> <code style="font-family:monospace,monospace">Direction: ESME (through VLR and HLR) => SGSN / MSC</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">No need to reverse, this is correct. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I meant the order in the braces. As it is now I read that the ESME sends to the VLR and then to the HLR. So I'd favor:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  ESME (through HLR and VLR) -> MSC</pre></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11836/2/common/chapters/gsup.adoc@545">Patch Set #2, Line 545:</a> <code style="font-family:monospace,monospace">short message delivery from SGSN (PS domain) / MSC (CS domain) to an ESME</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">"... from SGSN (PS domain) / MSC (CS domain) to an ESME ..." […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">It would help to put 'an earlier' in there:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  This message is used to indicate a negative result of an earlier<br>  MO short message delivery</pre><p style="white-space: pre-wrap; word-wrap: break-word;">well ... and I still think an "MO message delivery" is clear and no need to explain its direction again.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11836/2/common/chapters/gsup.adoc@1094">Patch Set #2, Line 1094:</a> <code style="font-family:monospace,monospace">Relay Layer)_ has an unique _message reference (see 8.2.3)_, that</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">> 8.2.3 in this document? then use a <<foo>> link. 8.2.3 in 04.11? […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">To unclutter and remove uncertainties, I'd still rather have each reference in one place and complete, instead of spreading them with words in-between.</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: 2 </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: Wed, 21 Nov 2018 13:34:21 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>