<p><a href="https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/22611">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/osmo-gsm-manuals/+/22611/1/common/chapters/trx_if.adoc">File common/chapters/trx_if.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/osmo-gsm-manuals/+/22611/1/common/chapters/trx_if.adoc@198">Patch Set #1, Line 198:</a> <code style="font-family:monospace,monospace">----</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Ack</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Well, we're dealing with a *human readable* protocol here. I would love to go for something more convenient, so we would not need to mess with sprintf() / sscanf(), but re-designing the TRXC is out of scope so far. I just wanted this command to look a bit more readable.</p><p style="white-space: pre-wrap; word-wrap: break-word;">With timeslot and subslot it's logical that the former goes first, from the hierarchical point of view. With the training sequence and set we could assume that set goes first, but the more parameters we add, the harder it gets to grasp what corresponds to what without looking at the protocol reference.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Regarding parsing complexity, it's just a longer format string for sscanf(), nothing special. If you both insist, I would change to plain numbers.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/22611">change 22611</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/osmo-gsm-manuals/+/22611"/><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-Change-Id: I621e46edac5d748472fd645f81876116904e260c </div>
<div style="display:none"> Gerrit-Change-Number: 22611 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 02 Feb 2021 15:49:25 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Comment-In-Reply-To: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>