rlenferink commented on code in PR #446:
URL: https://github.com/apache/celix/pull/446#discussion_r977993872


##########
libs/utils/include_deprecated/linked_list.h:
##########
@@ -32,63 +32,57 @@
 #include "celix_errno.h"
 #include "exports.h"
 
-#ifdef ADD_CELIX_DEPRECATED_WARNING
-#define CELIX_DEPRECATED_ATTR __attribute__ ((deprecated))
-#else
-#define CELIX_DEPRECATED_ATTR
-#endif
-
 #ifdef __cplusplus
 extern "C" {
 #endif
-typedef struct linked_list_entry *linked_list_entry_pt; //TODO make deprecated
-typedef struct linked_list *linked_list_pt; //TODO make deprecated
+typedef struct linked_list_entry *linked_list_entry_pt;

Review Comment:
   There was a note with `TODO make deprecated`. Why only mark the 
`linked_list_pt` as deprecated and not this one?



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