<p style="white-space: pre-wrap; word-wrap: break-word;">One of the questions where I don't have a clear feeling about is whether we should prefix also the BBV FSM with bssgp2_ instead of bssgp_.</p><p style="white-space: pre-wrap; word-wrap: break-word;">It's not like we had a FSM before, so there is no need to separate a new from an old implementatin in prefix.  But then, mayb it makes sense to prefix all symbols introduced around the same time/generation of code with the same prefix?</p><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/21598">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/libosmocore/+/21598/3/src/gb/bssgp_bvc_fsm.c">File src/gb/bssgp_bvc_fsm.c:</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/libosmocore/+/21598/3/src/gb/bssgp_bvc_fsm.c@4">Patch Set #3, Line 4:</a> <code style="font-family:monospace,monospace">/* FIXME: This should probably be moved to libosmogb? */</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">it's already inlibosmogb right?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Ack</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmocore/+/21598/3/src/gb/bssgp_bvc_fsm.c@70">Patch Set #3, Line 70:</a> <code style="font-family:monospace,monospace">             uint32_t advertised;</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">not important, but these can really be uint16_t then.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">AS you can see, today's spec already utilizes all 16 of those  16 bits.  Wait for Rel16 or later defining more bits, and you will be happy we have used an uint32_t here from the start :P</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmocore/+/21598/3/src/gb/bssgp_bvc_fsm.c@283">Patch Set #3, Line 283:</a> <code style="font-family:monospace,monospace">         osmo_fsm_inst_state_chg(fi, BSSGP_BVCFSM_S_UNBLOCKED, T1_SECS, T1);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Not required, but you probably want to move to using osmo_tdef structures and FSM APIs at some point […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">There's actually a related FIXME in Line 25, thanks.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/21598">change 21598</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/libosmocore/+/21598"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Icbe8e4f03b68fd73b8eae95f6f6cccd4fa9af95a </div>
<div style="display:none"> Gerrit-Change-Number: 21598 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 08 Dec 2020 12:24:50 +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"> Gerrit-MessageType: comment </div>