This is an automated email from the ASF dual-hosted git repository.
pnoltes pushed a commit to branch feature/pubsub_utils_test_fix
in repository https://gitbox.apache.org/repos/asf/celix.git
The following commit(s) were added to refs/heads/feature/pubsub_utils_test_fix
by this push:
new 3616649 Adds a missing )
3616649 is described below
commit 36166496f10b0ea242e79e1a52b2dc3ffe715c33
Author: Pepijn Noltes <[email protected]>
AuthorDate: Wed Jun 10 21:17:09 2020 +0200
Adds a missing )
---
.../pubsub_utils/gtest/src/PubSubSerializationProviderTestSuite.cc | 2 +-
bundles/pubsub/pubsub_utils/src/pubsub_serialization_provider.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git
a/bundles/pubsub/pubsub_utils/gtest/src/PubSubSerializationProviderTestSuite.cc
b/bundles/pubsub/pubsub_utils/gtest/src/PubSubSerializationProviderTestSuite.cc
index 2522bbf..db39759 100644
---
a/bundles/pubsub/pubsub_utils/gtest/src/PubSubSerializationProviderTestSuite.cc
+++
b/bundles/pubsub/pubsub_utils/gtest/src/PubSubSerializationProviderTestSuite.cc
@@ -65,7 +65,7 @@ TEST_F(PubSubSerializationProviderTestSuite,
FindSerializationServices) {
size_t nrEntries = pubsub_serializationProvider_nrOfEntries(provider);
EXPECT_EQ(5, nrEntries);
size_t nrOfInvalidEntries =
pubsub_serializationProvider_nrOfInvalidEntries(provider);
- EXPECT_EQ(3, nrOfInvalidEntries); //note 4 invalid, because
garbage.descriptor is never added (cannot extract msgFqn)
+ EXPECT_EQ(3, nrOfInvalidEntries); //note 3 invalid, because
garbage.descriptor is never added (cannot extract msgFqn)
auto* services = celix_bundleContext_findServices(ctx.get(),
PUBSUB_MESSAGE_SERIALIZATION_SERVICE_NAME);
EXPECT_EQ(5, celix_arrayList_size(services)); //3 valid, 5 invalid
diff --git a/bundles/pubsub/pubsub_utils/src/pubsub_serialization_provider.c
b/bundles/pubsub/pubsub_utils/src/pubsub_serialization_provider.c
index 99e6d6b..02902be 100644
--- a/bundles/pubsub/pubsub_utils/src/pubsub_serialization_provider.c
+++ b/bundles/pubsub/pubsub_utils/src/pubsub_serialization_provider.c
@@ -266,7 +266,7 @@ static dyn_message_type*
pubsub_serializationProvider_parseDfiDescriptor(pubsub_
static bool
pubsub_serializationProvider_alreadyAddedEntry(pubsub_serialization_provider_t*
provider, pubsub_serialization_entry_t* entry) {
for (int i = 0; i <
celix_arrayList_size(provider->serializationSvcEntries); ++i) {
pubsub_serialization_entry_t *visit =
celix_arrayList_get(provider->serializationSvcEntries, i);
- if (celix_utils_stringEquals(visit->readFromEntryPath,
entry->readFromEntryPath) {
+ if (celix_utils_stringEquals(visit->readFromEntryPath,
entry->readFromEntryPath)) {
return true;
}
}