emacs-elpa-diffs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[nongnu] elpa/annotate acca7ab1f5 03/17: - substituted custom prompt wit


From: ELPA Syncer
Subject: [nongnu] elpa/annotate acca7ab1f5 03/17: - substituted custom prompt with a call to 'y-or-no-p'.
Date: Fri, 25 Feb 2022 14:57:55 -0500 (EST)

branch: elpa/annotate
commit acca7ab1f503c749f23ecdd5d4504b1ca6d55619
Author: cage <cage@invalid>
Commit: cage <cage@invalid>

    - substituted custom prompt with a call to 'y-or-no-p'.
---
 annotate.el | 24 +++++++-----------------
 1 file changed, 7 insertions(+), 17 deletions(-)

diff --git a/annotate.el b/annotate.el
index e230e617bf..0701468171 100644
--- a/annotate.el
+++ b/annotate.el
@@ -1610,11 +1610,9 @@ annotation."
              (actual-data (mapcar %abbreviate-filename data)))
         (prin1 actual-data (current-buffer)))))
    ((file-exists-p annotate-file)
-    (let* ((confirm-message    "Delete annotations database file %S? [y/N] ")
+    (let* ((confirm-message    "Delete annotations database file %S? ")
            (delete-confirmed-p (or (not annotate-database-confirm-deletion)
-                                   (string= (read-from-minibuffer (format 
confirm-message
-                                                                           
annotate-file))
-                                            "y"))))
+                                   (y-or-n-p (format confirm-message 
annotate-file)))))
       (if delete-confirmed-p
           (condition-case err
               (delete-file annotate-file t)
@@ -3166,12 +3164,9 @@ code, always use load files from trusted sources!"
                     (read-file-name "Database file location: "))))
     (when (not (annotate-string-empty-p new-db))
       (if (file-exists-p new-db)
-          (let* ((confirm-message "Loading elisp file from untrusted source 
may results in severe security problems. Load %S? [y/N] ")
-                 (load-file-confirmed (if force-load
-                                          t
-                                        (string= (read-from-minibuffer (format 
confirm-message
-                                                                               
new-db))
-                                                 "y"))))
+          (let* ((confirm-message "Loading elisp file from untrusted source 
may results in severe security problems. Load %S?")
+                 (load-file-confirmed (or force-load
+                                          (y-or-n-p (format confirm-message 
new-db)))))
             (if load-file-confirmed
                 (progn
                   (setf annotate-file new-db)
@@ -3294,14 +3289,9 @@ their personal database."
                                    (file-exists-p filename)))
                                annotations)))
   (let* ((confirm-message    (concat "Importing databases from untrusted 
source may cause severe "
-                                     "security issues, continue? [y/N] "))
+                                     "security issues, continue?"))
          (import-confirmed-p (or (not annotate-database-confirm-import)
-                                 ;; FIXME: There's no `%s' in `confirm-message'
-                                 ;; to make use of the `annotate-file' arg?
-                                 ;; FIXME: Use `y-or-n-p' or `yes-or-no-p'!
-                                 (string= (read-string (format confirm-message
-                                                                        
annotate-file))
-                                          "y"))))
+                                 (y-or-n-p confirm-message))))
     (when import-confirmed-p
       (let* ((imported-db-name (read-file-name "Choose the database to import: 
"))
              (imported-db      (remove-non-existing-files (deserialize-db 
imported-db-name)))



reply via email to

[Prev in Thread] Current Thread [Next in Thread]