<p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/23785">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-bsc/+/23785/1/include/osmocom/bsc/bts.h">File include/osmocom/bsc/bts.h:</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-bsc/+/23785/1/include/osmocom/bsc/bts.h@674">Patch Set #1, Line 674:</a> <code style="font-family:monospace,monospace">int gsm_net_set_system_infos();</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">we have a lot of function declarations without args that omit 'void' ... […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">https://www.c-programming-simple-steps.com/what-is-void.html:<br>"In C, if you don’t specify the parameters of a function, it can accept any number of parameters of any type.  The way to define a function, that does not accept parameters in C is to use the keyword void as the only element in the parameters list."</p><p style="white-space: pre-wrap; word-wrap: break-word;">We already had issues in the past with not setting params as "void" which I had to fix, where wrong params or unused params end up being passed and the compiler doesn't warn, making code confusing. Having some functions not compling to it doesn't mean we should pay attention on new functions (and I'm sure I also introduce this kind of stuff myself from time to time).</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/23785">change 23785</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-bsc/+/23785"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I2222e029fc225152e124ed1e8887f1ffd4a107ef </div>
<div style="display:none"> Gerrit-Change-Number: 23785 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </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: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 20 Apr 2021 08:23:21 +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: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>