<p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/20408">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-bts/+/20408/1/include/osmo-bts/nm_bts_sm_fsm.h">File include/osmo-bts/nm_bts_sm_fsm.h:</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-bts/+/20408/1/include/osmo-bts/nm_bts_sm_fsm.h@40">Patch Set #1, Line 40:</a> <code style="font-family:monospace,monospace">nm_bts_sm_fsm</code></p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Why not nm_bts_site_manager?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">This looks good too, even better than my proposal. Since this symbol is not going to be used often, I don't think we need to shorten it. I vote for readability in this situation.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Or xyz? nm_bts_sm_fsm looks good for me.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">'xyz' looks like irrelevant irony. I am not trying to throw a stick between your legs, just posting my concerns / ideas, so let's avoid that. 'nm_bts_sm_fsm' may look confusing, because I haven't seen such abbreviation in the specs. (neither in the existing code), and it may be hard to find the relation for other people looking at this code. Moreover, 'SM' usually stands for 'Short Message' in TS 04.11 and the related core network specs. This is why I suggest to rename it.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/20408">change 20408</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-bts/+/20408"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bts </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Iab2d17c45c9642860cd2d5d523c1baae24502243 </div>
<div style="display:none"> Gerrit-Change-Number: 20408 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-CC: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 05 Oct 2020 10:35:08 +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: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>