Small url-cache update.

* lisp/url/url-cache.el (url-cache-expire-time): New option.
(url-cache-expired): Rewrite.

diff --git a/lisp/url/ChangeLog b/lisp/url/ChangeLog
index 9bc9d3110f816a18d1aeaa96708718e9b72581cb..f19c639e930af481ef8eb9895130e448256d16c6 100644 (file)
--- a/lisp/url/ChangeLog
+++ b/lisp/url/ChangeLog
@@ -1,3 +1,8 @@
+2010-09-23  Julien Danjou  <julien@danjou.info>
+
+       * url-cache.el (url-cache-expire-time): New option.
+       (url-cache-expired): Rewrite.
+
 2010-09-19  Julien Danjou  <julien@danjou.info>
 
        * url-cache.el (url-fetch-from-cache): New function.

diff --git a/lisp/url/url-cache.el b/lisp/url/url-cache.el
index 28d0af761eae63985742a6c9dcd24f579d0d6242..834ad7c2530524e508e2800d6693de528181086a 100644 (file)
--- a/lisp/url/url-cache.el
+++ b/lisp/url/url-cache.el
@@ -32,6 +32,12 @@
   :type 'directory
   :group 'url-file)
 
+(defcustom url-cache-expire-time 3600
+  "Maximum time in seconds to keep the documents cached."
+  :version "24.1"
+  :type 'integer
+  :group 'url-cache)
+
 ;; Cache manager
 (defun url-cache-file-writable-p (file)
   "Follows the documentation of `file-writable-p', unlike `file-writable-p'."
@@ -186,21 +192,19 @@ Very fast if you have an `md5' primitive function, suitably fast otherwise."
   (insert-file-contents-literally fnam))
 
 ;;;###autoload
-(defun url-cache-expired (url mod)
-  "Return t if a cached file has expired."
-  (let* ((urlobj (if (vectorp url) url (url-generic-parse-url url)))
-        (type (url-type urlobj)))
-    (cond
-     (url-standalone-mode
-      (not (file-exists-p (url-cache-create-filename url))))
-     ((string= type "http")
-      t)
-     ((member type '("file" "ftp"))
-      (if (or (equal mod '(0 0)) (not mod))
-         t
-       (or (> (nth 0 mod) (nth 0 (current-time)))
-           (> (nth 1 mod) (nth 1 (current-time))))))
-     (t nil))))
+(defun url-cache-expired (url &optional expire-time)
+  "Return t if a cached URL is more than EXPIRE-TIME old.
+If EXPIRE-TIME is not set, `url-cache-expire-time' is used instead."
+  (cond (url-standalone-mode
+        (not (file-exists-p (url-cache-create-filename url))))
+       (t (let ((cache-time (url-is-cached url)))
+            (if cache-time
+                (time-less-p
+                 (time-add
+                  (url-is-cached url)
+                  (seconds-to-time (or expire-time url-cache-expire-time)))
+                 (current-time))
+              t)))))
 
 (provide 'url-cache)