<p>Max has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/12246">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">cosmetic: use const pointer for bts_data<br><br>It's used several time for logging so let's call it once to make code<br>easier to follow.<br><br>Change-Id: Icfd9e5603a5d8701f487f17e9c0335d458e9e80b<br>---<br>M src/tbf_dl.cpp<br>M src/tbf_ul.cpp<br>2 files changed, 6 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/46/12246/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/tbf_dl.cpp b/src/tbf_dl.cpp</span><br><span>index dd24963..01331a6 100644</span><br><span>--- a/src/tbf_dl.cpp</span><br><span>+++ b/src/tbf_dl.cpp</span><br><span>@@ -1357,9 +1357,10 @@</span><br><span> </span><br><span> void gprs_rlcmac_dl_tbf::set_window_size()</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">-  uint16_t ws = egprs_window_size(bts->bts_data(), dl_slots());</span><br><span style="color: hsl(120, 100%, 40%);">+      const struct gprs_rlcmac_bts *b = bts->bts_data();</span><br><span style="color: hsl(120, 100%, 40%);">+ uint16_t ws = egprs_window_size(b, dl_slots());</span><br><span>      LOGPTBFDL(this, LOGL_INFO, "setting EGPRS DL window size to %u, base(%u) slots(%u) ws_pdch(%u)\n",</span><br><span style="color: hsl(0, 100%, 40%);">-              ws, bts->bts_data()->ws_base, pcu_bitcount(dl_slots()), bts->bts_data()->ws_pdch);</span><br><span style="color: hsl(120, 100%, 40%);">+                ws, b->ws_base, pcu_bitcount(dl_slots()), b->ws_pdch);</span><br><span>       m_window.set_ws(ws);</span><br><span> }</span><br><span> </span><br><span>diff --git a/src/tbf_ul.cpp b/src/tbf_ul.cpp</span><br><span>index 02f4ddb..f877484 100644</span><br><span>--- a/src/tbf_ul.cpp</span><br><span>+++ b/src/tbf_ul.cpp</span><br><span>@@ -593,8 +593,9 @@</span><br><span> </span><br><span> void gprs_rlcmac_ul_tbf::set_window_size()</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">- uint16_t ws = egprs_window_size(bts->bts_data(), ul_slots());</span><br><span style="color: hsl(120, 100%, 40%);">+      const struct gprs_rlcmac_bts *b = bts->bts_data();</span><br><span style="color: hsl(120, 100%, 40%);">+ uint16_t ws = egprs_window_size(b, ul_slots());</span><br><span>      LOGPTBFUL(this, LOGL_INFO, "setting EGPRS UL window size to %u, base(%u) slots(%u) ws_pdch(%u)\n",</span><br><span style="color: hsl(0, 100%, 40%);">-              ws, bts->bts_data()->ws_base, pcu_bitcount(ul_slots()), bts->bts_data()->ws_pdch);</span><br><span style="color: hsl(120, 100%, 40%);">+                ws, b->ws_base, pcu_bitcount(ul_slots()), b->ws_pdch);</span><br><span>       m_window.set_ws(ws);</span><br><span> }</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/12246">change 12246</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/12246"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-pcu </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: Icfd9e5603a5d8701f487f17e9c0335d458e9e80b </div>
<div style="display:none"> Gerrit-Change-Number: 12246 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Max <msuraev@sysmocom.de> </div>