summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>2019-08-19 19:34:09 +0300
committerLaurent Pinchart <laurent.pinchart@ideasonboard.com>2019-08-22 11:09:05 +0300
commitaf4304e497837963388e8ac4253c49cd6bd3ef54 (patch)
tree4088bc86a6589693876d1175aab41bacb9e6753e
parentd209b2d4b4352b21114e129d088f15ddb381dee4 (diff)
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 <laurent.pinchart@ideasonboard.com> Reviewed-by: Paul Elder <paul.elder@ideasonboard.com>
-rw-r--r--src/libcamera/process.cpp11
1 files changed, 8 insertions, 3 deletions
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) {