<p style="white-space: pre-wrap; word-wrap: break-word;">Are you planning to submit similar to wireshark?</p><p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/17718">View Change</a></p><p>5 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/17718/1/src/gsm_rlcmac.c">File src/gsm_rlcmac.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/osmo-pcu/+/17718/1/src/gsm_rlcmac.c@5337">Patch Set #1, Line 5337:</a> <code style="font-family:monospace,monospace">  M_CHOICE       (PacketChannelRequest_11b_t, Type,</code></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">I think you need to first parse Type, or are you doing it somewhere else? See for instance:<br>CSN_DESCR_BEGIN(IA_EGPRS_t)<br>  M_UINT       (IA_EGPRS_t,  UnionType ,  1 ),<br>  M_CHOICE     (IA_EGPRS_t, UnionType, IA_EGPRS_Choice, ElementsOf(IA_EGPRS_Choice)),<br>CSN_DESCR_END  (IA_EGPRS_t)</pre></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/17718/1/src/gsm_rlcmac.c@6319">Patch Set #1, Line 6319:</a> <code style="font-family:monospace,monospace">  struct bitvec bv = {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">If we have an API to allocate better use that, in order not to assume implementation details.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/17718/1/src/gsm_rlcmac.c@6325">Patch Set #1, Line 6325:</a> <code style="font-family:monospace,monospace">  switch (req->type) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">In decode functions afaik the data structure is memzeroes by default. I'd rather pass this type as a differnet param if needed, and assign it here.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/17718/1/src/gsm_rlcmac.c@6333">Patch Set #1, Line 6333:</a> <code style="font-family:monospace,monospace">        descr = CSNDESCR(EGPRS_PacketChannelRequest_t);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Better have one function for each initial csnStreamInit like we do with others.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/17718/1/tests/rlcmac/RLCMACTest.cpp">File tests/rlcmac/RLCMACTest.cpp:</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-pcu/+/17718/1/tests/rlcmac/RLCMACTest.cpp@481">Patch Set #1, Line 481:</a> <code style="font-family:monospace,monospace">   static const uint16_t GPRSPktCh11bReqs[] = {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Did you think about implementing encoding and using the struct and pass it to the encoder then to the decoder?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-pcu/+/17718">change 17718</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-pcu/+/17718"/><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-Change-Id: I96df3352856933c9140177b2801a2c71f4134183 </div>
<div style="display:none"> Gerrit-Change-Number: 17718 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Sat, 04 Apr 2020 12:46:09 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>