<p><a href="https://gerrit.osmocom.org/c/libosmocore/+/20332">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/libosmocore/+/20332/1/src/gsm/gad.c">File src/gsm/gad.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/libosmocore/+/20332/1/src/gsm/gad.c@250">Patch Set #1, Line 250:</a> <code style="font-family:monospace,monospace">    *gad = (struct osmo_gad){};</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">>> I think we ha that discussion before, and the language construct neels uses here should be equiva […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">hmm I see that memcmp() is unfortunately not a good choice to compare structs (and I knew that before, but comparing every single element is so dumb, I wish C allowed '==' on structs).</p><p style="white-space: pre-wrap; word-wrap: break-word;">Still I generally want to zero-initialize all struct members, regardless of whether padding is also zeroed or not. I usually choose struct member types such that their zero value represents "not set" / "not initialized" / a sane default value.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The GAD struct does not really need that, and we could discuss that, but it's not like leaving random data in it would improve anything in any noticeable way. So I'd rather not spend time on this aspect.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/20332">change 20332</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/libosmocore/+/20332"/><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-Change-Id: I7a9dd805a91b1ebb6353bde0cd169218acbf223c </div>
<div style="display:none"> Gerrit-Change-Number: 20332 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 05 Oct 2020 19:00:58 +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: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Comment-In-Reply-To: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>