[GitHub] [celix] pnoltes commented on a change in pull request #373: Updates C++ header only to C++17

[GitBox <gi...@apache.org>]

pnoltes commented on a change in pull request #373:
URL: https://github.com/apache/celix/pull/373#discussion_r725677204



##########
File path: libs/framework/include/celix/Constants.h
##########
@@ -211,4 +211,12 @@ namespace celix {
      * If not specified ".cache" is used.
      */
     constexpr const char * const FRAMEWORK_CACHE_DIR = CELIX_FRAMEWORK_FRAMEWORK_STORAGE;
+
+    /**
+     * @brief Celix framework environment property (named "CELIX_FRAMEWORK_WARN_FOR_MISSING_SERVICE_VERSION")
+     * which configures the Celix framework to print warning when service are registered without a version property.
+     *
+     * Default is true.
+     */
+    constexpr const char * const WARN_FOR_MISSING_SERVICE_VERSION = CELIX_FRAMEWORK_WARN_FOR_MISSING_SERVICE_VERSION;

Review comment:
       This entry can be removed, warning for missing service version is not added in this PR.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@celix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org