This is an automated email from the ASF dual-hosted git repository. pnoltes pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/celix.git
commit c6dadea04eea053a7a0ad35c89a1a46aae2d01f3 Author: Deedss <[email protected]> AuthorDate: Wed Apr 8 19:20:26 2026 +0000 Fix last remaining use of setRequired --- libs/framework/benchmark/src/DependencyManagerBenchmark.cc | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/libs/framework/benchmark/src/DependencyManagerBenchmark.cc b/libs/framework/benchmark/src/DependencyManagerBenchmark.cc index 5c9cea787..0a52a322c 100644 --- a/libs/framework/benchmark/src/DependencyManagerBenchmark.cc +++ b/libs/framework/benchmark/src/DependencyManagerBenchmark.cc @@ -17,8 +17,10 @@ * under the License. */ -#include <benchmark/benchmark.h> +#include "../../include/celix_dm_service_dependency.h" + #include "celix/FrameworkFactory.h" +#include <benchmark/benchmark.h> //note using c++ service for both the C and C++ benchmark, because this should not impact the performance. class IService { @@ -81,7 +83,7 @@ static void createAndDestroyComponentTest(benchmark::State& state, bool cTest) { auto* dep = celix_dmServiceDependency_create(); celix_dmServiceDependency_setService(dep, IService::NAME, nullptr, nullptr); - celix_dmServiceDependency_setRequired(dep, true); + celix_dmServiceDependency_setMinimalCardinality(dep, 1); celix_dmComponent_addServiceDependency(cmp, dep); celix_dependencyManager_addAsync(cMan, cmp);
