<p><a href="https://gerrit.osmocom.org/9281">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/9281/7/src/osmo-bsc/osmo_bsc_api.c">File src/osmo-bsc/osmo_bsc_api.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/9281/7/src/osmo-bsc/osmo_bsc_api.c@428">Patch Set #7, Line 428:</a> <code style="font-family:monospace,monospace">    LOGPFSML(conn->fi, LOGL_ERROR, "Assignment failure: BSSMAP: '%s' from RR: '%s'\n",</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Ok, this still looks stupid. But actually there are numerous callers passing in interesting cause values. Instead of killing all of those, I'd actually like to keep them around until I'm through refactoring the lchan assignment, because I might get around to reporting those cause values properly instead of dropping them on the floor...</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/9281/8/src/osmo-bsc/osmo_bsc_api.c">File src/osmo-bsc/osmo_bsc_api.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/9281/8/src/osmo-bsc/osmo_bsc_api.c@430">Patch Set #8, Line 430:</a> <code style="font-family:monospace,monospace">   osmo_fsm_inst_dispatch(conn->fi, GSCON_EV_RR_ASS_FAIL, &cause);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">does this qualify for the right direction?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/9281">change 9281</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/9281"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ib204b4a5272f9b7b60ca5f932cd8a4c857316270 </div>
<div style="display:none"> Gerrit-Change-Number: 9281 </div>
<div style="display:none"> Gerrit-PatchSet: 8 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 08 Jun 2018 03:49:33 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>