diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-05 09:22:15 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-05 09:22:15 -0500 |
| commit | 6737e1b603a0f54d72458bb6332cf1e8b1b26dee (patch) | |
| tree | 3fe8e868aca5eb49d738258012e7ebb82507e83d /org-drill.el | |
| parent | 9e25494338c7c3093d0c4f52783d75ebc0757514 (diff) | |
| download | org-drill-6737e1b603a0f54d72458bb6332cf1e8b1b26dee.tar.gz org-drill-6737e1b603a0f54d72458bb6332cf1e8b1b26dee.zip | |
refactor: drop Org <9.6 fallback in time-to-inactive-org-timestamp
The function had an Org-version branch with a legacy <9.6 path that
used (substring (cdr org-time-stamp-formats) 1 -1). Modern Org's
format-strings dropped the angle brackets that the substring slice
assumed, so the legacy path was both dead-code (unreachable under
the org>=9.6 dep floor we just declared) and silently buggy if it
ever did run.
Function is now a one-liner around the modern primitive.
Diffstat (limited to 'org-drill.el')
| -rw-r--r-- | org-drill.el | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/org-drill.el b/org-drill.el index dad46cb..75d2015 100644 --- a/org-drill.el +++ b/org-drill.el @@ -761,12 +761,7 @@ CMD is bound, or nil if it is not bound to a key." (defun org-drill-time-to-inactive-org-timestamp (time) "Convert TIME into org-mode timestamp." - (format-time-string - ;; TODO(deprecate-org-9.5): Remove backward-compat code. - (if (version<= "9.6" (org-version)) - (org-time-stamp-format t 'no-bracket) - (concat "[" (substring (cdr org-time-stamp-formats) 1 -1) "]")) - time)) + (format-time-string (org-time-stamp-format t 'no-bracket) time)) (defun org-drill-map-entries (func &optional scope drill-match &rest skip) "Like `org-map-entries', but only drill entries are processed." |