<p>Patch set 3:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/12714">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/12714/3/src/logging.c">File src/logging.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/12714/3/src/logging.c@60">Patch Set #3, Line 60:</a> <code style="font-family:monospace,monospace">log_info *osmo_log_info = NULL</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">to my knowledge this is not required at all.</p><p style="white-space: pre-wrap; word-wrap: break-word;">https://en.wikipedia.org/wiki/.bss<br>"Unix-like systems and Windows initialize the bss section to zero, allowing C and C++ statically-allocated variables initialized to values represented with all bits zero to be put in the bss segment"</p><p style="white-space: pre-wrap; word-wrap: break-word;">Also, from the C specification:</p><p style="white-space: pre-wrap; word-wrap: break-word;">In C code, any variable with static storage duration is defined to be initialized to 0 by the spec (Section 6.7.8 Initialization, paragraph 10):</p><p style="white-space: pre-wrap; word-wrap: break-word;">If an object that has static storage duration is not initialized explicitly, then:</p><ul><li>if it has pointer type, it is initialized to a null pointer;</li><li>if it has arithmetic type, it is initialized to (positive or unsigned) zero;</li><li>if it is an aggregate, every member is initialized (recursively) according to these rules;</li><li>if it is a union, the first named member is initialized (recursively) according to these rules.</li></ul></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12714">change 12714</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/12714"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ifd97c945f63cfe9850f025db7e69370866ea133e </div>
<div style="display:none"> Gerrit-Change-Number: 12714 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 29 Jan 2019 10:23:47 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>