Fix: tests: add `void` parameter to functions that take no parameters
authorSimon Marchi <simon.marchi@efficios.com>
Wed, 25 Mar 2020 22:39:39 +0000 (18:39 -0400)
committerJérémie Galarneau <jeremie.galarneau@efficios.com>
Thu, 26 Mar 2020 23:07:45 +0000 (19:07 -0400)
commit4d4b447e26c82c5f2cebe425e3bb63c7ac98faaa
tree6734313b6cf45c6f95dc2c72791fc408c97e1a58
parent16057fe164df71db1ae538781a91d2170ba50967
Fix: tests: add `void` parameter to functions that take no parameters

In C, empty parenthesis declare a function without a prorotype (without
specifying its parameters).  This is not the same as having a `void`
parameter, which declares a function which has no parameters.

It's safer to use the later, otherwise it makes it possible to
erroneously call the function with some arguments.

Change this `test_function` to add `void`.  It fixes diagnostics like:

  CC       userspace-probe-elf-binary.o
/home/simark/src/lttng-tools/tests/utils/testapp/userspace-probe-elf-binary/userspace-probe-elf-binary.c:12:33: error: no previous prototype for ‘test_function’ [-Werror=missing-prototypes]
   12 | void __attribute__ ((noinline)) test_function()
      |

Change-Id: Iceb7636e44d45f51889667ec76f2c04c032b5df8
Signed-off-by: Simon Marchi <simon.marchi@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
tests/regression/ust/multi-lib/multi-lib-test.c
tests/utils/testapp/userspace-probe-elf-binary/userspace-probe-elf-binary.c
This page took 0.024883 seconds and 4 git commands to generate.