From af4304e497837963388e8ac4253c49cd6bd3ef54 Mon Sep 17 00:00:00 2001 From: Laurent Pinchart Date: Mon, 19 Aug 2019 19:34:09 +0300 Subject: libcamera: process: Properly ignore unused result with gcc Casting the return value of a function to (void) doesn't ignore the unused result warning with gcc. Use a #pragma to fix this properly, to fix compilation with _FORTIFY_SOURCE. Fixes: df23ab95f3d7 ("libcamera: process: fix compilation on Chromium OS") Signed-off-by: Laurent Pinchart Reviewed-by: Paul Elder --- src/libcamera/process.cpp | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) (limited to 'src/libcamera/process.cpp') diff --git a/src/libcamera/process.cpp b/src/libcamera/process.cpp index ab716a9c..3b4d0f10 100644 --- a/src/libcamera/process.cpp +++ b/src/libcamera/process.cpp @@ -68,10 +68,15 @@ namespace { void sigact(int signal, siginfo_t *info, void *ucontext) { +#pragma GCC diagnostic push +#pragma GCC diagnostic ignored "-Wunused-result" + /* + * We're in a signal handler so we can't log any message, and we need + * to continue anyway. + */ char data = 0; - /* We're in a signal handler so we can't log any message, - * and we need to continue anyway. */ - (void)write(ProcessManager::instance()->writePipe(), &data, sizeof(data)); + write(ProcessManager::instance()->writePipe(), &data, sizeof(data)); +#pragma GCC diagnostic pop const struct sigaction &oldsa = ProcessManager::instance()->oldsa(); if (oldsa.sa_flags & SA_SIGINFO) { -- cgit v1.2.1