core: Added void to declarations of functions with empty argument list
authorMarius Zbihlei <marius.zbihlei@1and1.ro>
Mon, 2 Apr 2012 14:19:17 +0000 (17:19 +0300)
committerMarius Zbihlei <marius.zbihlei@1and1.ro>
Mon, 2 Apr 2012 14:19:17 +0000 (17:19 +0300)
commitabb01fb41068f9accc49f638b107abc12d83ef5e
tree89026683517afc053ab99dbdfb9815f763ad9b02
parentb3568cf5613890d71560a40444bc16ab78a65954
core: Added void  to declarations of functions with empty argument list

In C language, a declaration in the form int f(); is equivalent to int f(...);, thus being able to accept an indefinit number of parameters. With the -Wstrict-prototypes GCC options, these declarations are reported as "function declaration isn’t a prototype".
On some cases, this may trick the compiler into generating unoptimized code (like preparing to handle variadic argument list).
In all cases having a declaration int f() and a definition inf f(int) is missleading, even if standard compliant.
This is still Work in Progress. (maybe adding the -Wstrict-prototypes option to default is desireable)
37 files changed:
atomic_ops_init.h
basex.h
cfg/cfg_select.h
counters.h
daemonize.c
daemonize.h
dns_cache.h
dprint.h
dst_blacklist.c
dst_blacklist.h
endianness.h
flags.h
hash_func.c
lock_ops.c
lock_ops_init.h
main.c
nonsip_hooks.h
poll_types.h
pt.h
rand/fastrand.h
resolve.h
route.h
rpc_lookup.h
script_cb.h
sctp_options.h
sctp_server.h
select_buf.h
shm_init.h
socket_info.h
tcp_init.h
tcp_options.h
tcp_stats.h
timer.h
tls_hooks.h
tls_hooks_init.h
udp_server.h
ut.h