diff options
| author | Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com> | 2020-07-16 11:29:43 +0200 |
|---|---|---|
| committer | Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com> | 2020-09-14 20:48:41 +0200 |
| commit | 6103951f42886f9f9e9f1304f566456a9fc8bc53 (patch) | |
| tree | c52e10d48cb8b0f6bf4cc1f2cb40197f472889d8 /tests/core_hotunplug.c | |
| parent | 4b192984ec7e845a1ea49a385b7669642c3b3ec9 (diff) | |
tests/core_hotunplug: Explicitly ignore unused return values
Some return values are not useful and can be ignored. Wrap those cases
inside igt_ignore_warn(), not only to make sure compilers are happy but
also to clearly document our decisions.
Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>
Reviewed-by: Michał Winiarski <michal.winiarski@intel.com>
Diffstat (limited to 'tests/core_hotunplug.c')
| -rw-r--r-- | tests/core_hotunplug.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tests/core_hotunplug.c b/tests/core_hotunplug.c index a7dc4cf3..6cf56d04 100644 --- a/tests/core_hotunplug.c +++ b/tests/core_hotunplug.c @@ -261,7 +261,7 @@ static void set_filter_from_device(int fd) char path[PATH_MAX + 1]; igt_assert(igt_sysfs_path(fd, path, PATH_MAX)); - strncat(path, "/device", PATH_MAX - strlen(path)); + igt_ignore_warn(strncat(path, "/device", PATH_MAX - strlen(path))); igt_assert(realpath(path, dst)); igt_device_filter_free_all(); @@ -398,7 +398,7 @@ igt_main igt_fixture { post_healthcheck(&priv); - close(priv.fd.sysfs_bus); - close(priv.fd.sysfs_drv); + igt_ignore_warn(close(priv.fd.sysfs_bus)); + igt_ignore_warn(close(priv.fd.sysfs_drv)); } } |