PengZheng commented on code in PR #524: URL: https://github.com/apache/celix/pull/524#discussion_r1175144174
########## libs/framework/gtest/src/BundleArchiveWithErrorInjectionTestSuite.cc: ########## @@ -27,9 +27,9 @@ #include "asprintf_ei.h" //extern declarations for testing purposes. Note signatures are not correct, but that is not important for the test. -extern "C" celix_status_t bundleArchive_create(void); +extern "C" celix_status_t celix_bundleArchive_create(void); extern "C" celix_status_t manifest_create(void); -extern "C" celix_status_t bundleRevision_create(void); +extern "C" celix_status_t celix_bundleRevision_create(void); Review Comment: Now that the new celix api is used, why not just include corresponding 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
