branch: externals/denote
commit 20dc124e884b88e9371e3bfd019b3690e70916e7
Author: Jean-Philippe Gagné Guay <jeanphilippe...@gmail.com>
Commit: Jean-Philippe Gagné Guay <jeanphilippe...@gmail.com>

    Add comments to denote-generate-identifier-as-number
---
 denote.el | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/denote.el b/denote.el
index fd2b359ede..9d498a37bb 100644
--- a/denote.el
+++ b/denote.el
@@ -3072,13 +3072,21 @@ Else, use the first unused number starting from 1.
 
 This is a reference function for `denote-get-identifier-function'."
   (let ((denote-used-identifiers (or denote-used-identifiers 
(denote--get-all-used-ids))))
-    (cond ((and initial-identifier
+    (cond (;; Always use the supplied initial-identifier if possible,
+           ;; regardless of format.
+           (and initial-identifier
                 (not (gethash initial-identifier denote-used-identifiers)))
            initial-identifier)
-          ((and initial-identifier
+          (;; If the supplied initial-identifier is already used, but
+           ;; it has the right format, make is unique.
+           (and initial-identifier
                 (string-match-p "[1-9][0-9]*" initial-identifier))
            (denote--find-first-unused-id-as-number initial-identifier))
-          (t
+          (;; Else, the supplied initial-identifier is nil or it is
+           ;; already used or it does not match the supplied
+           ;; format. Ignore it and generate a valid identifier with
+           ;; the right format.
+           t
            (denote--find-first-unused-id-as-number "1")))))
 
 (defvar denote-command-prompt-history nil

Reply via email to