You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@celix.apache.org by "pnoltes (via GitHub)" <gi...@apache.org> on 2023/04/28 13:15:57 UTC

Re: [PR] [WIP]Some module layer fixes (celix)

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


##########
libs/framework/src/module.c:
##########
@@ -251,12 +251,12 @@ celix_status_t module_getGroup(module_pt module, const char **symbolicName) {
     return status;
 }
 
-celix_status_t module_getDescription(module_pt module, const char **symbolicName) {
+celix_status_t module_getDescription(module_pt module, const char **descriptoin) {
     celix_status_t status = CELIX_SUCCESS;
     if (module == NULL) {
         status = CELIX_ILLEGAL_ARGUMENT;
     } else {
-        *symbolicName = module->description;
+        *descriptoin = module->description;

Review Comment:
   nitpick: typo in `descriptoin` variable. 



-- 
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-unsubscribe@celix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org