From dfb05a1cf5479e6949d2413a633431c64d006ff2 Mon Sep 17 00:00:00 2001 From: David Herrmann Date: Tue, 4 Nov 2014 09:49:43 +0100 Subject: barrier: explicitly ignore return values of barrier_place() The barrier implementation tracks remote states internally. There is no need to check the return value of any barrier_*() function if the caller is not interested in the result. The barrier helpers only return the state of the remote side, which is usually not interesting as later calls to barrier_sync() will catch this, anyway. Shut up coverity by explicitly ignoring return values of barrier_place() if we're not interested in it. --- src/shared/barrier.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/shared') diff --git a/src/shared/barrier.h b/src/shared/barrier.h index c55e311344..d4ad2a419b 100644 --- a/src/shared/barrier.h +++ b/src/shared/barrier.h @@ -91,6 +91,6 @@ static inline bool barrier_is_aborted(Barrier *b) { } static inline bool barrier_place_and_sync(Barrier *b) { - barrier_place(b); + (void)barrier_place(b); return barrier_sync(b); } -- cgit v1.2.3-54-g00ecf