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

Reply via email to