You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@celix.apache.org by bp...@apache.org on 2015/11/11 22:35:01 UTC

celix git commit: CELIX-289: fix warnings

Repository: celix
Updated Branches:
  refs/heads/develop 4f6b0c08d -> 9ed4837b7


CELIX-289: fix warnings


Project: http://git-wip-us.apache.org/repos/asf/celix/repo
Commit: http://git-wip-us.apache.org/repos/asf/celix/commit/9ed4837b
Tree: http://git-wip-us.apache.org/repos/asf/celix/tree/9ed4837b
Diff: http://git-wip-us.apache.org/repos/asf/celix/diff/9ed4837b

Branch: refs/heads/develop
Commit: 9ed4837b79459b58eeb534eaf8918cc7c0058064
Parents: 4f6b0c0
Author: Bjoern Petri <bp...@apache.org>
Authored: Wed Nov 11 22:34:33 2015 +0100
Committer: Bjoern Petri <bp...@apache.org>
Committed: Wed Nov 11 22:34:33 2015 +0100

----------------------------------------------------------------------
 utils/private/test/celix_threads_test.cpp | 1 +
 utils/private/test/hash_map_test.cpp      | 8 ++++----
 2 files changed, 5 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/celix/blob/9ed4837b/utils/private/test/celix_threads_test.cpp
----------------------------------------------------------------------
diff --git a/utils/private/test/celix_threads_test.cpp b/utils/private/test/celix_threads_test.cpp
index 29e4475..620b42b 100644
--- a/utils/private/test/celix_threads_test.cpp
+++ b/utils/private/test/celix_threads_test.cpp
@@ -288,6 +288,7 @@ static void * thread_test_func_exit(void *) {
 	*pi = 666;
 
 	celixThread_exit(pi);
+	return NULL;
 }
 
 static void * thread_test_func_detach(void *) {

http://git-wip-us.apache.org/repos/asf/celix/blob/9ed4837b/utils/private/test/hash_map_test.cpp
----------------------------------------------------------------------
diff --git a/utils/private/test/hash_map_test.cpp b/utils/private/test/hash_map_test.cpp
index 024655d..2051942 100644
--- a/utils/private/test/hash_map_test.cpp
+++ b/utils/private/test/hash_map_test.cpp
@@ -167,10 +167,10 @@ TEST(hash_map, create){
 	CHECK(map != NULL);
 	LONGS_EQUAL(0, map->size);
 	// This fails on windows due to dllimport providing a proxy for exported functions.
-	CHECK_EQUAL(hashMap_equals, map->equalsKey);
-	CHECK_EQUAL(hashMap_equals, map->equalsValue);
-	CHECK_EQUAL(hashMap_hashCode, map->hashKey);
-	CHECK_EQUAL(hashMap_hashCode, map->hashValue);
+	CHECK_EQUAL(&hashMap_equals, map->equalsKey);
+	CHECK_EQUAL(&hashMap_equals, map->equalsValue);
+	CHECK_EQUAL(&hashMap_hashCode, map->hashKey);
+	CHECK_EQUAL(&hashMap_hashCode, map->hashValue);
 }
 
 TEST(hash_map, size){