pnoltes commented on a change in pull request #392:
URL: https://github.com/apache/celix/pull/392#discussion_r791031367
##########
File path: libs/framework/src/bundle_context.c
##########
@@ -76,7 +76,7 @@ celix_status_t bundleContext_create(framework_pt framework,
celix_framework_logg
}
}
-
+ //FIXME: context == NULL?
Review comment:
Do you also know if there is a good way to check (unit test) if the code
correctly handles NULL returns from malloc/calloc?
IMO this is the primary reason we dropped checking malloc/calloc returns.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
