Since failure() is [[noreturn]], we can invert the conditional so that
we don't need an else. This silences a -Wunused-parameter warning.
Signed-off-by: Alejandro Colomar <alx@kernel.org>
* first group member might be just a normal user.
* --marekm
*/
-#if defined(FIRST_MEMBER_IS_ADMIN)
+#if !defined(FIRST_MEMBER_IS_ADMIN)
+ failure();
+#endif
if (gr->gr_mem[0] == NULL)
failure();
if (strcmp(gr->gr_mem[0], myname) != 0)
failure();
-#else /* ! FIRST_MEMBER_IS_ADMIN */
- failure();
-#endif
}
}