PengZheng commented on code in PR #524:
URL: https://github.com/apache/celix/pull/524#discussion_r1175158534


##########
libs/framework/src/bundle_archive_private.h:
##########
@@ -41,9 +41,9 @@ extern "C" {
 
 /**
  * @brief Create bundle archive.
- *
+ * @note Symbol exported for testing purposes.
  */
-celix_status_t bundleArchive_create(celix_framework_t* fw, const char 
*archiveRoot, long id, const char *location, bundle_archive_pt *bundle_archive);
+CELIX_FRAMEWORK_EXPORT celix_status_t 
celix_bundleArchive_create(celix_framework_t* fw, const char *archiveRoot, long 
id, const char *location, bundle_archive_pt *bundle_archive);

Review Comment:
   We could use the object library for unit test. Thus it should be unnecessary.



##########
libs/framework/src/bundle_private.h:
##########
@@ -53,11 +54,14 @@ celix_status_t
 celix_bundle_createFromArchive(celix_framework_t *framework, bundle_archive_pt 
archive, celix_bundle_t **bundleOut);
 
 /**
- * Get the bundle archive.
+ * @brief Get the bundle archive.
+ *
+ * @note Symbol export is needed for unit tests.
+ *
  * @param[in] bundle The bundle.
  * @return The bundle archive.
  */
-bundle_archive_t *celix_bundle_getArchive(const celix_bundle_t *bundle);
+CELIX_FRAMEWORK_EXPORT bundle_archive_t *celix_bundle_getArchive(const 
celix_bundle_t *bundle);

Review Comment:
   Let's use object target for unit test and remove it.



-- 
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

Reply via email to