PengZheng commented on code in PR #476: URL: https://github.com/apache/celix/pull/476#discussion_r1115547188
########## libs/framework/include/bundle_archive.h: ########## @@ -77,6 +77,21 @@ FRAMEWORK_EXPORT celix_status_t bundleArchive_setPersistentState(bundle_archive_ FRAMEWORK_EXPORT celix_status_t bundleArchive_getPersistentState(bundle_archive_pt archive, bundle_state_e *state); +/** + * @brief Return the last modified time of the bundle archive. + * + * The last modified time is based on the last modified time of the bundle archives MANIFEST.MF file. + * + * If the bundle archive cache directory does not exist, lastModified will be set to 0. + * + * @param[in] archive The bundle archive. + * @parar[in] alreadyLocked If true, the bundle archive lock is already acquired. + * @param[out] lastModified The last modified time of the bundle archive. + * @return CELIX_SUCCESS if the last modified time could be retrieved, CELIX_ILLEGAL_ARGUMENT if the last modified Review Comment: The only possible error is `CELIX_FILE_IO_EXCEPTION` instead of `CELIX_ILLEGAL_ARGUMENT`? -- 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