<p><a href="https://gerrit.osmocom.org/13402">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/13402/1/src/coding_scheme.c">File src/coding_scheme.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/13402/1/src/coding_scheme.c@131">Patch Set #1, Line 131:</a> <code style="font-family:monospace,monospace">/* FIXME: take into account padding and special cases of commanded MCS (MCS-6-9 and MCS-5-7) */</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">commanded MCS? do you mean demanded_mcs?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/13402/1/src/coding_scheme.c@136">Patch Set #1, Line 136:</a> <code style="font-family:monospace,monospace"> OSMO_ASSERT(NUM_SCHEMES - MCS1 == 9);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">That should be actually a compile-time assert.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Have a look at "_Static_assert": https://en.cppreference.com/w/c/language/_Static_assert</p><p style="white-space: pre-wrap; word-wrap: break-word;">It seems for c++ "static_assert" is used instead.</p><p style="white-space: pre-wrap; word-wrap: break-word;">May be worth adding an OSMO_STATIC_ASSERT() macro in libosmocore to make it automatically c/c++ compatible, as explained in https://stackoverflow.com/a/54993033</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/13402">change 13402</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/13402"/><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: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ia73baeefee7a58834f0fc50e3b8bf8d5e3eb7815 </div>
<div style="display:none"> Gerrit-Change-Number: 13402 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 25 Mar 2019 16:37:56 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>