PengZheng commented on code in PR #536: URL: https://github.com/apache/celix/pull/536#discussion_r1183418388
########## libs/utils/include/celix_errno.h: ########## @@ -178,6 +178,7 @@ CELIX_UTILS_EXPORT const char* celix_strerror(celix_status_t status); */ #define CELIX_ENOMEM CELIX_ERROR_MAKE(CELIX_FACILITY_CERRNO,ENOMEM) +#define CELIX_EAGAIN CELIX_ERROR_MAKE(CELIX_FACILITY_CERRNO,EAGAIN) Review Comment: If we make it explicit that `celix_status_t` contains all errno, then this is unnecessary. Of course we should also extend `celix_strerror`. -- 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
