From 9824ded5dded6f1d25ba7c71d959d56ac2f10e7e Mon Sep 17 00:00:00 2001 From: Glenn Morris Date: Tue, 11 Dec 2007 05:49:36 +0000 Subject: [PATCH] (url-history-save-history): Use url-make-private-file and with-temp-buffer. --- lisp/url/ChangeLog | 9 +++++++++ lisp/url/url-history.el | 38 ++++++++++++++++++-------------------- 2 files changed, 27 insertions(+), 20 deletions(-) diff --git a/lisp/url/ChangeLog b/lisp/url/ChangeLog index 9d86733ab98..a1b86af13ae 100644 --- a/lisp/url/ChangeLog +++ b/lisp/url/ChangeLog @@ -1,3 +1,12 @@ +2007-12-11 Glenn Morris + + * url.el (url-configuration-directory): Make it a defcustom. + + * url-util.el (url-make-private-file): New function. + * url-cookie.el (url-cookie-write-file): + * url-history.el (url-history-save-history): + Use url-make-private-file and with-temp-buffer. + 2007-12-06 Glenn Morris * url-file.el, url-mailto.el: Remove directory part from filenames diff --git a/lisp/url/url-history.el b/lisp/url/url-history.el index 605ffa0145f..bf4acf5d046 100644 --- a/lisp/url/url-history.el +++ b/lisp/url/url-history.el @@ -112,29 +112,28 @@ to run the `url-history-setup-save-timer' function manually." (puthash (if (vectorp url) (url-recreate-url url) url) time url-history-hash-table)) +(autoload 'url-make-private-file "url-util") + (defun url-history-save-history (&optional fname) "Write the global history file into `url-history-file'. The type of data written is determined by what is in the file to begin with. If the type of storage cannot be determined, then prompt the user for what type to save as." (interactive) - (or fname (setq fname (expand-file-name url-history-file))) - (unless (file-directory-p (file-name-directory fname)) - (condition-case nil - (make-directory (file-name-directory fname)) - (error nil))) - (cond - ((not url-history-changed-since-last-save) nil) - ((not (file-writable-p fname)) - (message "%s is unwritable." fname)) - (t - (let ((make-backup-files nil) - (version-control nil) - (require-final-newline t)) - (with-current-buffer (get-buffer-create " *url-tmp*") - (erase-buffer) - (let ((count 0)) - (maphash (lambda (key value) + (when url-history-changed-since-last-save + (or fname (setq fname (expand-file-name url-history-file))) + (if (condition-case nil + (progn + (url-make-private-file fname) + nil) + (error t)) + (message "Error accessing history file `%s'" fname) + (let ((make-backup-files nil) + (version-control nil) + (require-final-newline t) + (count 0)) + (with-temp-buffer + (maphash (lambda (key value) (while (string-match "[\r\n]+" key) (setq key (concat (substring key 0 (match-beginning 0)) (substring key (match-end 0) nil)))) @@ -153,9 +152,8 @@ user for what type to save as." ;; (/ count 4))) ;; (goto-char (point-max)) (insert "\n") - (write-file fname)) - (kill-buffer (current-buffer)))))) - (setq url-history-changed-since-last-save nil)) + (write-file fname))) + (setq url-history-changed-since-last-save nil)))) (defun url-have-visited-url (url) (url-do-setup) -- 2.39.2