pnoltes commented on code in PR #591:
URL: https://github.com/apache/celix/pull/591#discussion_r1279831769


##########
libs/utils/include/celix_threads.h:
##########
@@ -104,40 +104,45 @@ CELIX_UTILS_EXPORT celix_status_t 
celixThreadMutex_tryLock(celix_thread_mutex_t
 CELIX_UTILS_EXPORT celix_status_t celixThreadMutex_unlock(celix_thread_mutex_t 
*mutex);
 
 /**
- * Opaque type. See celix_mutex_locker_new() for details.
+ * @brief Lock guard for mutex.
  */
-typedef void celix_mutex_locker_t;
+typedef struct celix_mutex_lock_guard {

Review Comment:
   :+1:, I like this clean and clear.



-- 
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