PengZheng commented on code in PR #476: URL: https://github.com/apache/celix/pull/476#discussion_r1138712791
########## libs/framework/src/celix_bundle_cache.h: ########## @@ -100,7 +97,42 @@ celix_bundleCache_createArchive(celix_framework_t *fw, long id, const char *loca * - CELIX_ILLEGAL_ARGUMENT If the cache is invalid * - CELIX_FILE_IO_EXCEPTION If the cache cannot be opened or read. */ -celix_status_t celix_bundleCache_delete(celix_bundle_cache_t *cache); +celix_status_t celix_bundleCache_deleteCacheDir(celix_bundle_cache_t *cache); + +/** + * @brief Find if the there is already a bundle cache for the provided bundle zip location and if this is true + * return the bundle id for the bundle cache entry. + * + * @param fw The framework. + * @param location The location of the bundle zip to find the id for. + * @return The bundle id or -1 if not found. + */ +long celix_bundleCache_findBundleIdForLocation(celix_framework_t *fw, const char *location); + +/** + * @brief Find if the there is already a bundle cache for the provided bundle id. + * @param fw The framework. + * @param bndId The bundle id to find the bundle cache for. + * @return Whether the bundle id is already used in a bundle cache entry. + */ +bool celix_bundleCache_isBundleIdAlreadyUsed(celix_framework_t *fw, long bndId); + +/** + * Clean existing cache dir and create the bundle archives cache for all the bundles configured for starting and + * installed. + * + * For bundle ids, the first bundle will have CELIX_FRAMEWORK_BUNDLE_ID+1 and the next CELIX_FRAMEWORK_BUNDLE_ID+2 etc. + * + * CELIX_AUTO_START_0, CELIX_AUTO_START_1, CELIX_AUTO_START_2, CELIX_AUTO_START_3, CELIX_AUTO_START_4, + * CELIX_AUTO_START_5, CELIX_AUTO_START_6 and lastly CELIX_AUTO_INSTALL. + * + * The returned hash map uses the bundle id as key and the bundle archive as value. + * Review Comment: ```suggestion ``` There is no hash map returned. -- 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