|
From: | Carsten Dominik |
Subject: | Re: [Orgmode] [PATCH 2/3] Serialize publishing project cache with `puthash' expressions. |
Date: | Sun, 23 May 2010 07:56:13 +0200 |
On May 22, 2010, at 4:10 PM, David Maus wrote:
--- lisp/org-publish.el | 29 +++++++++++++++++------------ 1 files changed, 17 insertions(+), 12 deletions(-) diff --git a/lisp/org-publish.el b/lisp/org-publish.el index fefd50d..dc94f7d 100644 --- a/lisp/org-publish.el +++ b/lisp/org-publish.el @@ -902,15 +902,18 @@ If FREE-CACHE, empty the cache." (let ((cache-file (org-publish-cache-get ":cache-file:"))) (unless cache-file- (error "%s" "Cannot find cache-file name in `org-publish- write-cache-file'"))+ (error+ "%s" "Cannot find cache-file name in `org-publish-write- cache-file'"))(with-temp-file cache-file (let ((print-level nil) - (print-length nil)) - (insert - "(setq org-publish-cache\n "- (replace-regexp-in-string "\\([^\\ \t]\"\\) \\([^ \t]\\)" "\\1\n\ \2"- (format "%S" org-publish-cache)) - ")\n"))) + (print-length nil)) + (maphash (lambda (k v) + (insert + (format (concat "(puthash %S " + (if (or (listp v) (symbolp v)) + "'" "") + "%S org-publish-cache)\n") k v))) + org-publish-cache))) (when free-cache (org-publish-reset-cache))))
I think it would be more future-proof to put the statement initializing the hash also into the cache file, so that the caches file then reads
(setq org-publish-cache (make-hash...)) (puthash.....) ............. Or maybe I am wrong? Sebastian, do you have a comment? Also, for long tables it might be more efficient to write the hash out as a list, and then loop over it to fill the cache again.Not sure how large projects get out there, though..... So this is not important now.
- Carsten
(defun org-publish-initialize-cache (project-name) @@ -929,16 +932,18 @@ and return it." (unless (and org-publish-cache (string= (org-publish-cache-get ":project:") project-name)) - (when org-publish-cache (org-publish-reset-cache)) (let* ((cache-file (concat (expand-file-name org-publish-timestamp-directory) project-name ".cache")) (cexists (file-exists-p cache-file))) - (if cexists (load-file cache-file)) - (unless org-publish-cache - (setq org-publish-cache - (make-hash-table :test 'equal :weakness nil :size 100)) + + (when org-publish-cache + (org-publish-reset-cache)) + (setq org-publish-cache + (make-hash-table :test 'equal :weakness nil :size 100)) + + (if cexists (load-file cache-file) (org-publish-cache-set ":project:" project-name) (org-publish-cache-set ":cache-file:" cache-file)) (unless cexists (org-publish-write-cache-file nil)))) -- 1.7.1 _______________________________________________ Emacs-orgmode mailing list Please use `Reply All' to send replies to the list. address@hidden http://lists.gnu.org/mailman/listinfo/emacs-orgmode
- Carsten
[Prev in Thread] | Current Thread | [Next in Thread] |