summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSam Clegg <sbc@chromium.org>2014-09-08 16:45:57 -0700
committerSam Clegg <sbc@chromium.org>2014-09-08 16:48:04 -0700
commitccd8ba9bbaecb392a2587e2a046286af3f3cccef (patch)
tree88aa2c867c144a731d5bf5964e99d8b7ecff4b8d
parent1e71354e34f360b19285ca108cd07903122a0a62 (diff)
downloadlibgit2-ccd8ba9bbaecb392a2587e2a046286af3f3cccef.tar.gz
Fix warnings in thread-utils.h when building without -DTHREADSAFE=ON
The compiler was generating a bunch of warnings for git_mutex_init and git_mutex_lock when GIT_THREADS was not defined (i.e. when not using -DTHREADSAFE=ON). Also remove an unused variable from tests/path/core.c.
-rw-r--r--src/thread-utils.h4
-rw-r--r--tests/path/core.c2
2 files changed, 2 insertions, 4 deletions
diff --git a/src/thread-utils.h b/src/thread-utils.h
index 5511a5117..8deedce81 100644
--- a/src/thread-utils.h
+++ b/src/thread-utils.h
@@ -183,8 +183,8 @@ GIT_INLINE(int64_t) git_atomic64_add(git_atomic64 *a, int64_t addend)
/* Pthreads Mutex */
#define git_mutex unsigned int
-#define git_mutex_init(a) 0
-#define git_mutex_lock(a) 0
+static int git_mutex_init(git_mutex* mutex) { GIT_UNUSED(mutex); return 0; }
+static int git_mutex_lock(git_mutex* mutex) { GIT_UNUSED(mutex); return 0; }
#define git_mutex_unlock(a) (void)0
#define git_mutex_free(a) (void)0
diff --git a/tests/path/core.c b/tests/path/core.c
index be63e309b..45f54df29 100644
--- a/tests/path/core.c
+++ b/tests/path/core.c
@@ -16,8 +16,6 @@ static void test_make_relative(
void test_path_core__make_relative(void)
{
- git_buf buf = GIT_BUF_INIT;
-
test_make_relative("foo.c", "/path/to/foo.c", "/path/to", 0);
test_make_relative("bar/foo.c", "/path/to/bar/foo.c", "/path/to", 0);
test_make_relative("foo.c", "/path/to/foo.c", "/path/to/", 0);