aboutsummaryrefslogtreecommitdiff
path: root/test-from-git
diff options
context:
space:
mode:
authorCraig Jennings <c@cjennings.net>2026-05-05 04:29:28 -0500
committerCraig Jennings <c@cjennings.net>2026-05-05 04:29:28 -0500
commitb2cc4d1645b378043a971906b5ebc34d3c92b544 (patch)
treebbb8c0fa4eeca0f5fc4a3df76d8aac7337bc0a57 /test-from-git
parent0cc022912ad65a67ee89bca11ac035b1018d5ec9 (diff)
downloadorg-drill-b2cc4d1645b378043a971906b5ebc34d3c92b544.tar.gz
org-drill-b2cc4d1645b378043a971906b5ebc34d3c92b544.zip
fix: guard org-drill-smart-reschedule cond against nil days-ahead
The function takes `days-ahead' as &optional, but the schedule cond called `(= 0 days-ahead)' and `(cl-minusp days-ahead)' before any type-guard, so passing nil crashed with a wrong-type-argument error. Today's two callers (the rating-confirmation flow and the org-drill-relearn-item helper) always pass a number, so this was latent — but a third caller relying on the documented &optional shape would hit it immediately. Switched the cond to require numberp before the value comparisons, and the default branch now falls back to the algorithm-computed next-interval when days-ahead is nil. That matches the intent implied by the optional signature and the docstring.
Diffstat (limited to 'test-from-git')
0 files changed, 0 insertions, 0 deletions