pnoltes commented on a change in pull request #373: URL: https://github.com/apache/celix/pull/373#discussion_r739814781
########## File path: libs/framework/include/module.h ########## @@ -89,6 +89,10 @@ FRAMEWORK_EXPORT celix_array_list_t *module_getDependents(module_pt module); FRAMEWORK_EXPORT celix_status_t module_getGroup(module_pt module, const char **group); +FRAMEWORK_EXPORT celix_status_t module_getName(module_pt module, const char **name); Review comment: For module there is no `celix_module_t`, the reason it that `module`, `archive`, `manifest`, `service_reference`, etc are not accessible though the celix_bundle_context.h header. Everything accessible though the celix_bundle_context.h I would like to keep in the next major Celix release, the rest will be removed or be placed in "private" headers. -- 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