You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@celix.apache.org by pn...@apache.org on 2016/06/26 07:55:44 UTC
[1/2] celix git commit: CELIX-364: Fixes bug with incorrect destroy
function
Repository: celix
Updated Branches:
refs/heads/develop 6de29e9c5 -> 8c4e037db
CELIX-364: Fixes bug with incorrect destroy function
Project: http://git-wip-us.apache.org/repos/asf/celix/repo
Commit: http://git-wip-us.apache.org/repos/asf/celix/commit/13d5d97b
Tree: http://git-wip-us.apache.org/repos/asf/celix/tree/13d5d97b
Diff: http://git-wip-us.apache.org/repos/asf/celix/diff/13d5d97b
Branch: refs/heads/develop
Commit: 13d5d97ba003444e3d3a41a247bcf45487d04055
Parents: 6de29e9
Author: Pepijn Noltes <pe...@gmail.com>
Authored: Sun Jun 26 09:54:05 2016 +0200
Committer: Pepijn Noltes <pe...@gmail.com>
Committed: Sun Jun 26 09:54:05 2016 +0200
----------------------------------------------------------------------
.../private/src/dm_dependency_manager_impl.c | 15 ++++++---------
1 file changed, 6 insertions(+), 9 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/celix/blob/13d5d97b/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 3e67f8c..2da30e5 100644
--- a/dependency_manager/private/src/dm_dependency_manager_impl.c
+++ b/dependency_manager/private/src/dm_dependency_manager_impl.c
@@ -119,14 +119,11 @@ celix_status_t dependencyManager_getInfo(dm_dependency_manager_pt manager, dm_de
}
void dependencyManager_destroyInfo(dm_dependency_manager_pt manager, dm_dependency_manager_info_pt info) {
-
- unsigned int i = 0;
- for(;i<arrayList_size(info->components);i++){
- dm_component_info_pt cmpinfo = (dm_component_info_pt)arrayList_get(info->components,0);
- component_destroyComponentInfo(cmpinfo);
- }
- arrayList_destroy(info->components);
-
+ unsigned int i = 0;
+ for (; i < arrayList_size(info->components); i += 1) {
+ dm_component_info_pt cmpinfo = (dm_component_info_pt)arrayList_get(info->components, i);
+ component_destroyComponentInfo(cmpinfo);
+ }
+ arrayList_destroy(info->components);
free(info);
-
}
[2/2] celix git commit: CELIX-360: Removes enforcing coverity for
every build
Posted by pn...@apache.org.
CELIX-360: Removes enforcing coverity for every build
Project: http://git-wip-us.apache.org/repos/asf/celix/repo
Commit: http://git-wip-us.apache.org/repos/asf/celix/commit/8c4e037d
Tree: http://git-wip-us.apache.org/repos/asf/celix/tree/8c4e037d
Diff: http://git-wip-us.apache.org/repos/asf/celix/diff/8c4e037d
Branch: refs/heads/develop
Commit: 8c4e037db02f63796b10389df1ac409bcfe42b56
Parents: 13d5d97
Author: Pepijn Noltes <pe...@gmail.com>
Authored: Sun Jun 26 09:55:16 2016 +0200
Committer: Pepijn Noltes <pe...@gmail.com>
Committed: Sun Jun 26 09:55:16 2016 +0200
----------------------------------------------------------------------
.travis.yml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/celix/blob/8c4e037d/.travis.yml
----------------------------------------------------------------------
diff --git a/.travis.yml b/.travis.yml
index d1f6729..62af5fe 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -25,7 +25,7 @@ env:
- COVERITY_SCAN_TOKEN="iomLSuaE8KOZLDog-KK7Ug"
- COVERITY_SCAN_BUILD_URL="https://scan.coverity.com/scripts/travisci_build_coverity_scan.sh"
- COVERITY_SCAN_BUILD="curl -s $COVERITY_SCAN_BUILD_URL | sed 's/https:\/\/scan.coverity.com\/builds/https:\/\/scan.coverity.com\/builds?project=Apache+Celix/g' | bash"
- - COVERITY_SCAN_ALWAYS_ON="y"
+ - COVERITY_SCAN_ALWAYS_ON="n"
matrix:
include: