tags 670836 = patch thanks On 15/06/12 01:41, Steven Chamberlain wrote: > [...] failed due to a -Wunused-but-set-variable > that I didn't notice before in some FreeBSD-specific code.
Ah, now I see why; that compiler option just got re-enabled. I notice that a fix for this was added to src/platform-linux.cc by upstream, so here is an updated, simpler patch to imitate that for src/platform-freebsd.cc. Thanks, Regards, -- Steven Chamberlain ste...@pyro.eu.org
From: Steven Chamberlain <ste...@pyro.eu.org> Subject: Fix a -Wunused-but-set-variable in some FreeBSD-specific code Bug-Debian: http://bugs.debian.org/670836 Index: libv8-3.10.8.16/src/platform-freebsd.cc =================================================================== --- libv8-3.10.8.16.orig/src/platform-freebsd.cc 2012-06-15 02:10:25.000000000 +0100 +++ libv8-3.10.8.16/src/platform-freebsd.cc 2012-06-15 02:13:24.318133457 +0100 @@ -554,6 +554,7 @@ ASSERT(result == 0); result = pthread_mutex_init(&mutex_, &attrs); ASSERT(result == 0); + USE(result); } virtual ~FreeBSDMutex() { pthread_mutex_destroy(&mutex_); }
