rlenferink commented on a change in pull request #373:
URL: https://github.com/apache/celix/pull/373#discussion_r744272031
##########
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.
Makes sense. How about usage of `module_t*` vs `module_pt` ?
--
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
