celix-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From pnol...@apache.org
Subject celix git commit: CELIX-308: Fix clang warning (promoted to error with -Werror)
Date Fri, 27 Nov 2015 15:34:07 GMT
Repository: celix
Updated Branches:
  refs/heads/develop e5acd9069 -> f99256497


CELIX-308: Fix clang warning (promoted to error with -Werror)


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

Branch: refs/heads/develop
Commit: f992564971e782eb93c433195e650274912bbfed
Parents: e5acd90
Author: Pepijn Noltes <pepijnnoltes@gmail.com>
Authored: Fri Nov 27 16:33:34 2015 +0100
Committer: Pepijn Noltes <pepijnnoltes@gmail.com>
Committed: Fri Nov 27 16:33:34 2015 +0100

----------------------------------------------------------------------
 dependency_manager/private/src/dm_dependency_manager_impl.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/celix/blob/f9925649/dependency_manager/private/src/dm_dependency_manager_impl.c
----------------------------------------------------------------------
diff --git a/dependency_manager/private/src/dm_dependency_manager_impl.c b/dependency_manager/private/src/dm_dependency_manager_impl.c
index 511ef1e..24245b7 100644
--- a/dependency_manager/private/src/dm_dependency_manager_impl.c
+++ b/dependency_manager/private/src/dm_dependency_manager_impl.c
@@ -67,7 +67,7 @@ celix_status_t dependencyManager_add(dm_dependency_manager_pt manager, dm_compon
 }
 
 celix_status_t depedencyManager_removeAllComponents(dm_dependency_manager_pt manager) {
-	celix_status_t status;
+	celix_status_t status = CELIX_SUCCESS;
 
 	unsigned int i=0;
 	unsigned int size = arrayList_size(manager->components);
@@ -75,7 +75,7 @@ celix_status_t depedencyManager_removeAllComponents(dm_dependency_manager_pt
man
 	for(;i<size;i++){
 		dm_component_pt cmp = arrayList_get(manager->components, i);
 		printf("Stopping comp %s\n", component_getName(cmp));
-		status += component_stop(cmp);
+		component_stop(cmp);
 	}
 
 	while (!arrayList_isEmpty(manager->components)) {


Mime
View raw message