From 6737e1b603a0f54d72458bb6332cf1e8b1b26dee Mon Sep 17 00:00:00 2001 From: Craig Jennings Date: Tue, 5 May 2026 09:22:15 -0500 Subject: 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. --- org-drill.el | 7 +------ 1 file changed, 1 insertion(+), 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." -- cgit v1.2.3