Change in osmo-bsc[master]: update neighbor ARFCNs on startup and config changes
gerrit-no-reply at lists.osmocom.org
Tue Apr 20 08:23:21 UTC 2021
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/23785 )
Change subject: update neighbor ARFCNs on startup and config changes
Patch Set 2:
PS1, Line 674: int gsm_net_set_system_infos();
> we have a lot of function declarations without args that omit 'void' ... […]
"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."
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).
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/23785
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Owner: neels <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-CC: fixeria <vyanitskiy at sysmocom.de>
Gerrit-Comment-Date: Tue, 20 Apr 2021 08:23:21 +0000
Comment-In-Reply-To: neels <nhofmeyr at sysmocom.de>
Comment-In-Reply-To: pespin <pespin at sysmocom.de>
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the gerrit-log