PengZheng commented on a change in pull request #399:
URL: https://github.com/apache/celix/pull/399#discussion_r816547783



##########
File path: libs/framework/include/celix_bundle_context.h
##########
@@ -771,6 +771,12 @@ typedef struct celix_service_use_options {
      * and the bundle owning the service will also be provided to the callback.
      */
     void (*useWithOwner)(void *handle, void *svc, const celix_properties_t 
*props, const celix_bundle_t *svcOwner) OPTS_INIT;
+    /**
+     * @brief Call the provided callbacks from the caller thread directly if 
set, otherwise the callbacks will be called from the Celix event loop (most 
likely indirectly).
+     * Note that using blocking service in the Celix event loop is generally a 
bad idea, which should be avoided if possible.
+     */
+#define CELIX_SERVICE_USE_DIRECT          (1)
+    int flags;

Review comment:
       Ouch! It's fixed.




-- 
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-unsubscr...@celix.apache.org

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


Reply via email to