<p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/17965">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/17965/3/library/RLCMAC_EncDec.cc">File library/RLCMAC_EncDec.cc:</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-ttcn3-hacks/+/17965/3/library/RLCMAC_EncDec.cc@297">Patch Set #3, Line 297:</a> <code style="font-family:monospace,monospace">fprintf(stderr, "RLMAC: clone: hdr_bytes=%u extra_bits=%u (length_bytes=%u)\n", hdr_bytes, extra_bits, length_bytes);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">do we really want unconditional prints to stderr in non-error paths? same applies to more messages d […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">They are not errors, as in not supported. Regarding using TTCN3 log facilitites: indeed, they should be used, but while debugging some crash I wasn't able to find the logging generated by it (maybe it doesn't print synchronously?), so I ended up using fprintf(stderr). Since god knows what will happen after hitting one of those fprintf() (as in crash), let's use fprintf(stderr) to make sure we log something before potentially crashing.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Anyway that will be stripped at some point later. For now I'd go for merging this so we have the entire placeholders working even if only it encodes Ul Headter type and decode Dl Header type3, but that's enough to get all the tests in shape for later additions.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/17965">change 17965</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-ttcn3-hacks/+/17965"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ttcn3-hacks </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I2bc4f877a5e17c57ffa8cf05565dc8593b45aae8 </div>
<div style="display:none"> Gerrit-Change-Number: 17965 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 28 Apr 2020 17:04:06 +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: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>