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 2021/12/24 11:01:20 UTC

[celix] branch master updated (3030ad3 -> 70d6219)

This is an automated email from the ASF dual-hosted git repository.

pnoltes pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/celix.git.


    from 3030ad3  Merge pull request #385 from jermus67/feature/filter_attributes_support
     add 63f13c5  Updates C++ properties to support string_view if C++17 is used.
     add de6fbfe  Applies use of string_view for all C++17 headers
     add 4d659e3  Replaces string_view returns to std::string
     new 70d6219  Merge pull request #382 from apache/feature/apply_use_stringview

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 libs/framework/include/celix/Bundle.h              |   7 +-
 libs/framework/include/celix/BundleContext.h       |  53 +++++-----
 libs/framework/include/celix/ServiceRegistration.h |  16 +--
 .../include/celix/ServiceRegistrationBuilder.h     |  24 +----
 libs/framework/include/celix/TrackerBuilders.h     |  10 +-
 libs/framework/include/celix/Trackers.h            |  28 ++---
 libs/framework/include/celix/UseServiceBuilder.h   |   6 +-
 libs/utils/gtest/CMakeLists.txt                    |  13 ++-
 libs/utils/gtest/src/CxxPropertiesTestSuite.cc     |  77 +++++++++++++-
 libs/utils/gtest/src/CxxUtilsTestSuite.cc          |  12 ++-
 libs/utils/include/celix/Filter.h                  |   9 +-
 libs/utils/include/celix/Properties.h              | 113 ++++++++++++++++++++-
 libs/utils/include/celix/Utils.h                   |  14 +--
 13 files changed, 284 insertions(+), 98 deletions(-)

[celix] 01/01: Merge pull request #382 from apache/feature/apply_use_stringview

Posted by pn...@apache.org.
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 70d6219ee48814e9f737b231d521da76e382b4d8
Merge: 3030ad3 4d659e3
Author: Pepijn Noltes <pe...@gmail.com>
AuthorDate: Fri Dec 24 11:59:56 2021 +0100

    Merge pull request #382 from apache/feature/apply_use_stringview
    
    Feature/apply use stringview

 libs/framework/include/celix/Bundle.h              |   7 +-
 libs/framework/include/celix/BundleContext.h       |  53 +++++-----
 libs/framework/include/celix/ServiceRegistration.h |  16 +--
 .../include/celix/ServiceRegistrationBuilder.h     |  24 +----
 libs/framework/include/celix/TrackerBuilders.h     |  10 +-
 libs/framework/include/celix/Trackers.h            |  28 ++---
 libs/framework/include/celix/UseServiceBuilder.h   |   6 +-
 libs/utils/gtest/CMakeLists.txt                    |  13 ++-
 libs/utils/gtest/src/CxxPropertiesTestSuite.cc     |  77 +++++++++++++-
 libs/utils/gtest/src/CxxUtilsTestSuite.cc          |  12 ++-
 libs/utils/include/celix/Filter.h                  |   9 +-
 libs/utils/include/celix/Properties.h              | 113 ++++++++++++++++++++-
 libs/utils/include/celix/Utils.h                   |  14 +--
 13 files changed, 284 insertions(+), 98 deletions(-)

diff --cc libs/utils/include/celix/Filter.h
index d552c7b,dacd6b2..ab00ec9
--- a/libs/utils/include/celix/Filter.h
+++ b/libs/utils/include/celix/Filter.h
@@@ -160,9 -135,8 +161,9 @@@ namespace celix 
          [[nodiscard]] bool empty() const {
              return cFilter == nullptr;
          }
 +
      private:
-         static std::shared_ptr<celix_filter_t> createFilter(const std::string& filterStr) {
+         static std::shared_ptr<celix_filter_t> createFilter(std::string_view filterStr) {
              if (filterStr.empty()) {
                  return nullptr;
              }