<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 style="white-space: pre-wrap; word-wrap: break-word;">do we really want unconditional prints to stderr in non-error paths? same applies to more messages down below.</p><p style="white-space: pre-wrap; word-wrap: break-word;">If they're not errors, they should go to stdout.  And if they are just for debugging the code, they should be default-off and enabled by some kind of environment variable or function to call for enablign it dynamically.</p><p style="white-space: pre-wrap; word-wrap: break-word;">And in general, it might be the best choice to use the TITAN logging framework to integrate with the other logging in titan native code.  This way it can be enabled/disabled via normal titan config files.</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-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 16:47:54 +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>