Suppress -Wunused-result warning about write().
authorNathan Bossart <[email protected]>
Wed, 6 Dec 2023 23:16:57 +0000 (17:16 -0600)
committerNathan Bossart <[email protected]>
Wed, 6 Dec 2023 23:16:57 +0000 (17:16 -0600)
pg_test_fsync's signal_cleanup() intentionally ignores the write()
result since there's not much we could do about it, but certain
compilers make that harder than it ought to be.

This was missed in commit 52e98d4502.

Reviewed-by: Tristan Partin, Peter Eisentraut
Discussion: https://postgr.es/m/20231206161839.GA2828158%40nathanxps13

src/bin/pg_test_fsync/pg_test_fsync.c

index f109aa5717456c03e5f0386ad534ff4931b931bc..54bc0767f18552d1bbaa6c5d82e892a5f1b8d595 100644 (file)
@@ -598,11 +598,14 @@ test_non_sync(void)
 static void
 signal_cleanup(SIGNAL_ARGS)
 {
+   int         rc;
+
    /* Delete the file if it exists. Ignore errors */
    if (needs_unlink)
        unlink(filename);
    /* Finish incomplete line on stdout */
-   write(STDOUT_FILENO, "\n", 1);
+   rc = write(STDOUT_FILENO, "\n", 1);
+   (void) rc;                  /* silence compiler warnings */
    _exit(1);
 }